applied patch from joakim that cleaned up some jsp, thanks!

git-svn-id: https://svn.apache.org/repos/asf/maven/archiva/trunk@442771 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/archiva-webapp/src/main/webapp/WEB-INF/jsp/decorators/default.jsp b/archiva-webapp/src/main/webapp/WEB-INF/jsp/decorators/default.jsp
index bc1c0505ce6058ca02ad74bd2ddcee6509676b92..3616e319b00d12ea5c8bcddd9349800563ccb050 100644 (file)
--- a/archiva-webapp/src/main/webapp/WEB-INF/jsp/decorators/default.jsp
+++ b/archiva-webapp/src/main/webapp/WEB-INF/jsp/decorators/default.jsp
@@ -59,8 +59,8 @@
 
 <div id="breadcrumbs">
   <div class="xleft">
-    <ww:url id="loginUrl" action="login" namespace="/" includeParams="none"/>
-    <ww:url id="registerUrl" action="register" namespace="/" includeParams="none"/>
+    <ww:url id="loginUrl" action="login" method="input" namespace="/" includeParams="none"/>
+    <ww:url id="registerUrl" action="register" method="input" namespace="/" includeParams="none"/>
 
     <ww:if test="${sessionScope.authStatus != true}">
       <ww:a href="%{loginUrl}">Login</ww:a> - <ww:a href="%{registerUrl}">Register</ww:a>

diff --git a/archiva-webapp/src/main/webapp/WEB-INF/jsp/login.jsp b/archiva-webapp/src/main/webapp/WEB-INF/jsp/login.jsp
index fa7a1f484d46a5f283030f31bdb3150a8268a66d..1b0ec284fc48afb33b38d8c8073961e7e5140c38 100644 (file)
--- a/archiva-webapp/src/main/webapp/WEB-INF/jsp/login.jsp
+++ b/archiva-webapp/src/main/webapp/WEB-INF/jsp/login.jsp
@@ -27,50 +27,46 @@
 
 <div id="contentArea">
   <div id="searchBox">
-    <div style="float: right">
-      <a href="#">Forgotten your Password?</a>
-
+    
+    <div id="results">
+      <%-- This is where the "Account Created Successfully" type message goes. --%>
+      <div class="success">
+           <ww:actionmessage />
+      </div>
+      <%-- This is where errors from the action and other non-form field specific errors appear. --%>
+      <div class="errors">
+       <ww:actionerror />
+      </div>
     </div>
 
-    <p>
-      <ww:actionmessage/>
-      <ww:actionerror/>
-    </p>
-
     <h2>Login</h2>
-    <ww:form action="login" method="post">
-      <table class="bodyTable">
-        <tr class="a">
-          <th>
-            Username
-          </th>
-          <td>
-            <ww:textfield name="username" size="30"/>
-          </td>
-        </tr>
-        <tr class="b">
-          <th>
-            Password
-          </th>
-          <td>
-            <ww:password name="password" size="20"/>
 
-          </td>
-        </tr>
-        <tr class="a">
-          <td></td>
-          <td>
-            <ww:submit value="Login"/>
-          </td>
-        </tr>
-      </table>
+       <%-- You don't need a table to wrap form elements in,
+            the ww:form creates the table, labels, context sensitive actionerrors, requirements indicators, etc...
+              - Joakim --%>
 
+    <ww:form action="login" method="post">
+      <ww:textfield label="Username" name="username" size="30" required="true" />
+      <ww:password  label="Password" name="password" size="20" required="true" />
+      <ww:submit value="Login"/>
     </ww:form>
-
-    <p>
-      <ww:url id="registerUrl" action="register"/>
-      Need an Account? <ww:a href="%{registerUrl}">Register!</ww:a>
-    </p>
+       
+    <ul class="tips">
+      <li>
+         Forgot your Username? 
+         <ww:url id="forgottenAccount" action="findAccount" />
+         <ww:a href="%{forgottenAccount}">Email me my account information.</ww:a>
+      </li>
+      <li>
+         Forgot your Password? 
+         <ww:url id="forgottenPassword" action="resetPassword" />
+         <ww:a href="%{forgottenPassword}">Request a password reset.</ww:a>
+      </li>
+      <li>
+        Need an Account?
+        <ww:url id="registerUrl" action="register" />
+        <ww:a href="%{registerUrl}">Register!</ww:a>
+    </ul>
   </div>
 </div>
 

diff --git a/archiva-webapp/src/main/webapp/WEB-INF/jsp/register.jsp b/archiva-webapp/src/main/webapp/WEB-INF/jsp/register.jsp
index b934041ff96c207618f3a79d99e8f78bf5ea31cf..bcb5d6ffcadf1f4c56c07f638337495fd3faa558 100644 (file)
--- a/archiva-webapp/src/main/webapp/WEB-INF/jsp/register.jsp
+++ b/archiva-webapp/src/main/webapp/WEB-INF/jsp/register.jsp
@@ -27,63 +27,31 @@
 
 <div id="contentArea">
   <div id="searchBox">
-    <p>
-      <ww:actionmessage/>
-      <ww:actionerror/>
-    </p>
+  
+    <div id="results">
+      <%-- This is where the "Account Created Successfully" type message goes. --%>
+      <div class="success">
+           <ww:actionmessage />
+      </div>
+      <%-- This is where errors from the action and other non-form field specific errors appear. --%>
+      <div class="errors">
+       <ww:actionerror />
+      </div>
+    </div>
 
     <h2>Register for an Account</h2>
-    <ww:form action="register" method="post">
-      <table class="bodyTable">
-        <tr class="b">
-          <th>
-            Username
-          </th>
-          <td>
-            <ww:textfield name="username" size="30"/>
-          </td>
-        </tr>
-        <tr class="a">
-          <th>
-            Password
-          </th>
-          <td>
-            <ww:password name="password" size="20"/>
-          </td>
-        </tr>
-        <tr class="b">
-          <th>
-            Confirm Password
-          </th>
-          <td>
-            <ww:password name="confirmPassword" size="20"/>
-          </td>
-
-        </tr>
-        <tr class="a">
-          <th>
-            Full Name
-          </th>
-          <td>
-            <ww:textfield name="fullName" size="30"/>
-          </td>
-        </tr>
 
-        <tr class="b">
-          <th>
-            Email
-          </th>
-          <td>
-            <ww:textfield name="email" size="50 "/>
-          </td>
-        </tr>
-        <tr class="a">
-          <td></td>
-          <td>
-            <ww:submit value="Register"/>
-          </td>
-        </tr>
-      </table>
+       <%-- You don't need a table to wrap form elements in,
+            the ww:form creates the table, labels, context sensitive actionerrors, requirements indicators, etc...
+              - Joakim --%>
+    
+    <ww:form action="register" method="post">
+               <ww:textfield label="Username"         name="username" size="30" required="true"/>
+               <ww:password  label="Password"         name="password" size="20" required="true"/>
+               <ww:password  label="Confirm Password" name="confirmPassword" size="20" required="true"/>
+        <ww:textfield label="Full Name"        name="fullName" size="30" required="true"/>
+        <ww:textfield label="Email Address"    name="email" size="50"    required="true"/>
+        <ww:submit    value="Register"/>
     </ww:form>
 
   </div>