From 2d2aed374aa94326f1d86cce9a234944426c3dda Mon Sep 17 00:00:00 2001
From: ChanMyeong <dece24@gmail.com>
Date: Wed, 12 Oct 2011 10:36:02 +0000
Subject: [PATCH] Member module mobile sign up UI clean.

git-svn-id: http://xe-core.googlecode.com/svn/branches/1.5.0@9617 201d5d3c-b55e-5fd7-737f-ddc643e51545
---
 modules/member/member.admin.view.php |  2 +-
 modules/member/tpl/css/msignup.css   |  7 +++-
 modules/member/tpl/signup_form.html  | 56 ++++++++++++----------------
 3 files changed, 29 insertions(+), 36 deletions(-)

diff --git a/modules/member/member.admin.view.php b/modules/member/member.admin.view.php
index bafe122a2..9a1498286 100644
--- a/modules/member/member.admin.view.php
+++ b/modules/member/member.admin.view.php
@@ -243,7 +243,7 @@
 								,zdate($memberInfo['birthday'], 'Y-m-d', false)
 								,$lang->cmd_delete);
 					}elseif($formInfo->name == 'find_account_question'){
-						$inputTag = '<select name="find_account_question" style="width:290px">%s</select><br />';
+						$inputTag = '<select name="find_account_question" style="width:290px">%s</select>';
 						$optionTag = array();
 						foreach($lang->find_account_question_items as $key=>$val){
 							if($key == $memberInfo['find_account_question']) $selected = 'selected="selected"';
diff --git a/modules/member/tpl/css/msignup.css b/modules/member/tpl/css/msignup.css
index 18ca5afb8..0798f4df1 100644
--- a/modules/member/tpl/css/msignup.css
+++ b/modules/member/tpl/css/msignup.css
@@ -32,13 +32,16 @@ input[type=radio]{width:13px;height:13px;margin:0;padding:0}
 .ff ul{margin:0 10px 10px 10px;padding:10px 0 0 0;list-style:none}
 .ff li{margin:0 0 5px 0;padding:0}
 .ff li p{color:#666}
+.ff li>label:first-child{display:block;font-weight:bold}
 .ff label+input[type=text],
 .ff label+input[type=password],
 .ff label+textarea,
-.ff label+select{display:block;width:96%;font-size:14px;margin:0 0 5px 0}
+.ff label+select,
+.ff label+select+input[type=text]{display:block;width:96%;font-size:14px;margin:0 0 5px 0}
 .ff label+input[type=text],
 .ff label+input[type=password],
-.ff label+textarea{padding:5px}
+.ff label+textarea,
+.ff label+select+input[type=text]{padding:5px}
 /* Button Area */
 .bna{text-align:center;padding:0 10px;margin:10px 0}
 .bna:after{content:"";display:block;clear:both}
diff --git a/modules/member/tpl/signup_form.html b/modules/member/tpl/signup_form.html
index 4a71c8e05..542033fe0 100644
--- a/modules/member/tpl/signup_form.html
+++ b/modules/member/tpl/signup_form.html
@@ -9,7 +9,7 @@
 	<div cond="$XE_VALIDATOR_MESSAGE" class="message {$XE_VALIDATOR_MESSAGE_TYPE}">
 		<p>{$XE_VALIDATOR_MESSAGE}</p>
 	</div>
-    <form ruleset="@insertMember" id="fo_insert_member" class="form" action="./" method="post" enctype="multipart/form-data">
+    <form ruleset="@insertMember" id="fo_insert_member" class="ff" action="./" method="post" enctype="multipart/form-data">
 		<input type="hidden" name="act" value="procMemberInsert" />
 		<!--@if(!$is_logged && $member_config->agreement)-->
 		<div class="agreement">
@@ -23,43 +23,33 @@
 		</div>
 		<!--@end-->
 		<ul>
-			<li>
-				<p class="q">{$identifierForm->title} <em style="color:red">*</em></p>
-				<p class="a">
-					<input type="text" name="{$identifierForm->name}" value="{$identifierForm->value}" />
-				</p>
+			<li> 
+				<label for="{$identifierForm->name}">{$identifierForm->title} <em style="color:red">*</em></label>
+				<input type="text" name="{$identifierForm->name}" id="{$identifierForm->name}" value="{$identifierForm->value}" />
 			</li>
-			<li>
-				<p class="q">{$lang->password} <em style="color:red">*</em></p>
-				<p class="a">
-					<input type="password" name="password" value=""/>
-				</p>
+			<li> 
+				<label for="password">{$lang->password} <em style="color:red">*</em></label>
+				<input type="password" name="password" id="password" value=""/>
 			</li>
-			<li>
-				<p class="q">{$lang->password2} <em style="color:red">*</em></p>
-				<p class="a">
-					<input type="password" name="password2" value=""/>
-				</p>
+			<li> 
+				<label for="password2">{$lang->password2} <em style="color:red">*</em></label>
+				<input type="password" name="password2" id="password2" value=""/>
 			</li>
-			<li loop="$formTags=>$formTag">
-				<p class="q">{$formTag->title}</p>
-				<div class="a" cond="$formTag->name != 'signature'">{$formTag->inputTag}</div>
-				<div class="a" cond="$formTag->name =='signature'">{$editor}</div>
+			<li loop="$formTags=>$formTag"> 
+				<label>{$formTag->title}</label>
+				<block cond="$formTag->name != 'signature'">{$formTag->inputTag}</block>
+				<block cond="$formTag->name =='signature'">{$editor}</block>
 			</li>
-			<li>
-				<p class="q">{$lang->allow_mailing}</p>
-				<p class="a">
-					<input type="radio" name="allow_mailing" id="mailingYes" value="Y" checked="checked"|cond="$member_info->allow_mailing == 'Y'"> <label for="mailingYes">{$lang->cmd_yes}</label>
-					<input type="radio" name="allow_mailing" id="mailingNo" value="N" checked="checked"|cond="$member_info->allow_mailing != 'Y'" > <label for="mailingNo">{$lang->cmd_no}</label>
-				</p>
+			<li> 
+				<label>{$lang->allow_mailing}</label>
+				<input type="radio" name="allow_mailing" id="mailingYes" value="Y" checked="checked"|cond="$member_info->allow_mailing == 'Y'"> <label for="mailingYes">{$lang->cmd_yes}</label>
+				<input type="radio" name="allow_mailing" id="mailingNo" value="N" checked="checked"|cond="$member_info->allow_mailing != 'Y'" > <label for="mailingNo">{$lang->cmd_no}</label>
 			</li>
-			<li>
-				<p class="q">{$lang->allow_message}</p>
-				<p class="a">
-					<block loop="$lang->allow_message_type=>$key,$val">
-						<input type="radio" name="allow_message" value="{$key}" checked="checked"|cond="$member_info->allow_message == $key" id="allow_{$key}" /> <label for="allow_{$key}">{$val}</label>
-					</block>
-				</p>
+			<li> 
+				<label>{$lang->allow_message}</label>
+				<block loop="$lang->allow_message_type=>$key,$val">
+					<input type="radio" name="allow_message" value="{$key}" checked="checked"|cond="$member_info->allow_message == $key" id="allow_{$key}" /> <label for="allow_{$key}">{$val}</label>
+				</block>
 			</li>
 		</ul>
 	<div class="bna">