From 7e201c01c06c2090fc22c6399f08c862b3127b35 Mon Sep 17 00:00:00 2001
From: Mariusz Dziemianowicz <mariusz.dziemianowicz@gmail.com>
Date: Sat, 12 Oct 2013 19:28:52 +0200
Subject: [PATCH] Worked on #25: Login/register pages

---
 public_html/media/css/auth.css     | 50 ++++++++++++++++++++++++++++++
 public_html/media/css/core.css     | 17 ++++++++++
 src/Controllers/AuthController.php |  2 ++
 src/Views/auth-login.phtml         | 17 ++++++----
 src/Views/auth-register.phtml      | 28 +++++++++++------
 5 files changed, 99 insertions(+), 15 deletions(-)
 create mode 100644 public_html/media/css/auth.css

diff --git a/public_html/media/css/auth.css b/public_html/media/css/auth.css
new file mode 100644
index 00000000..d0916ac0
--- /dev/null
+++ b/public_html/media/css/auth.css
@@ -0,0 +1,50 @@
+form.auth {
+	display: table;
+	border-collapse:separate;
+	border-spacing: 0 0.5em;
+	width: 400px;
+}
+
+form.auth div {
+	display: table-row;
+}
+
+form.auth label {
+	display: table-cell;
+	text-align: right;
+	float: left;
+	width: 144px;
+	margin: 5px 16px 0 0;
+}
+
+form.auth input {
+	display: table-cell;
+	font-family: inherit;
+	display: block;
+}
+
+form.auth input:not([type="submit"]) {
+	width: 156px;
+	padding: 5px;
+	border-width: 2px;
+	margin: 0;
+}
+
+form.auth input[type="submit"] {
+	margin: 0 auto;
+	background-color: #77f;
+	border: 0;
+	color: white;
+	font-weight: bold;
+	font-size: 1em;
+	padding: 8px 16px;
+}
+
+form.auth input[type="submit"]:hover {
+	background-color: #55f;
+}
+
+form.auth p {
+	text-align: center;
+	margin: 10px 0;
+}
diff --git a/public_html/media/css/core.css b/public_html/media/css/core.css
index 29b68317..e2697986 100644
--- a/public_html/media/css/core.css
+++ b/public_html/media/css/core.css
@@ -130,3 +130,20 @@ a:focus i[class*='icon-'],
 a:hover i[class*='icon-'] {
 	background-color: red;
 }
+
+.alert {
+	text-align: center;
+	padding: 10px;
+	border-style: solid;
+	border-width: 1px;
+}
+
+.alert-error {
+	border-color: #faa;
+	background-color: #fdd;
+}
+
+.alert-warning {
+	border-color: #ffa;
+	background-color: #ffd;
+}
diff --git a/src/Controllers/AuthController.php b/src/Controllers/AuthController.php
index a65b67bc..cd2625bc 100644
--- a/src/Controllers/AuthController.php
+++ b/src/Controllers/AuthController.php
@@ -12,6 +12,7 @@ class AuthController
 	*/
 	public function loginAction()
 	{
+		$this->context->stylesheets []= 'auth.css';
 		$this->context->subTitle = 'authentication form';
 
 		//check if already logged in
@@ -61,6 +62,7 @@ class AuthController
 	*/
 	public function registerAction()
 	{
+		$this->context->stylesheets []= 'auth.css';
 		$this->context->subTitle = 'registration form';
 
 		//check if already logged in
diff --git a/src/Views/auth-login.phtml b/src/Views/auth-login.phtml
index 13c45b61..3727e2c7 100644
--- a/src/Views/auth-login.phtml
+++ b/src/Views/auth-login.phtml
@@ -1,6 +1,11 @@
-<form action="<?php echo \Chibi\UrlHelper::route('auth', 'login') ?>" method="post">
+<form action="<?php echo \Chibi\UrlHelper::route('auth', 'login') ?>" class="auth" method="post">
+
 	<div>
-		<label for="user">User:</label>
+		<p>If you don't have an account yet,<br/><a href="<?php echo \Chibi\UrlHelper::route('auth', 'register'); ?>">click here</a> to create a new one.</p>
+	</div>
+
+	<div>
+		<label for="user">User name:</label>
 		<input id="user" name="user"/>
 	</div>
 
@@ -9,11 +14,11 @@
 		<input type="password" id="pass" name="pass"/>
 	</div>
 
+	<?php if (isset($this->context->transport->errorMessage)): ?>
+		<p class="alert alert-error">Error: <?php echo $this->context->transport->errorMessage ?></p>
+	<?php endif ?>
+
 	<div>
 		<input type="submit" value="Log in"/>
 	</div>
 </form>
-
-<?php if (isset($this->context->transport->errorMessage)): ?>
-	<p class="alert alert-error"><?php echo $this->context->transport->errorMessage ?></p>
-<?php endif ?>
diff --git a/src/Views/auth-register.phtml b/src/Views/auth-register.phtml
index b00d2986..7dd159d9 100644
--- a/src/Views/auth-register.phtml
+++ b/src/Views/auth-register.phtml
@@ -7,15 +7,14 @@
 		<p>After this, an admin will have to confirm your registration.</p>
 	<?php endif ?>
 <?php else: ?>
-	<form action="<?php echo \Chibi\UrlHelper::route('auth', 'register') ?>" method="post">
+	<form action="<?php echo \Chibi\UrlHelper::route('auth', 'register') ?>" class="auth" method="post">
 		<div>
-			<label for="user">User:</label>
-			<input id="user" name="user" value="<?php echo $this->context->suppliedUser ?>"/ placeholder="e.g. darth_vader" autocomplete="off">
+			<p>Registered users can view more content,<br/>upload files and add posts to favorites.</p>
 		</div>
 
 		<div>
-			<label for="pass">E-mail address<?php if ($this->context->transport->emailActivation) echo ' (required)' ?>:</label>
-			<input id="email" name="email" value="<?php echo $this->context->suppliedEmail ?>" placeholder="e.g. vader@empire.gov"/ autocomplete="off">
+			<label for="user">User name:</label>
+			<input id="user" name="user" value="<?php echo $this->context->suppliedUser ?>"/ placeholder="e.g. darth_vader" autocomplete="off">
 		</div>
 
 		<div>
@@ -28,12 +27,23 @@
 			<input type="password" id="pass" name="pass2" value="<?php echo $this->context->suppliedPass2 ?>" placeholder="e.g. <?php echo str_repeat('&#x25cf;', 8) ?>"/ autocomplete="off">
 		</div>
 
+		<div>
+			<label for="email">E-mail address<?php if ($this->context->transport->emailActivation) echo ' (required)' ?>:</label>
+			<input id="email" name="email" value="<?php echo $this->context->suppliedEmail ?>" placeholder="e.g. vader@empire.gov"/ autocomplete="off">
+		</div>
+
+		<div>
+			<p id="email-info" name="email-info">Your e-mail will be used to show your <a href="http://gravatar.com/">Gravatar</a>.<br/>Leave blank for random Gravatar.</p>
+		</div>
+
+		<?php if (isset($this->context->transport->errorMessage)): ?>
+		<div>
+			<p class="alert alert-error">Error: <?php echo $this->context->transport->errorMessage ?></p>
+		</div>
+		<?php endif ?>
+
 		<div>
 			<input type="submit" value="Register">
 		</div>
 	</form>
-
-	<?php if (isset($this->context->transport->errorMessage)): ?>
-		<p class="alert alert-error"><?php echo $this->context->transport->errorMessage ?></p>
-	<?php endif ?>
 <?php endif ?>