[LON-CAPA-cvs] cvs: loncom /auth lonlogin.pm

[bisitz]

bisitz		Tue Dec  1 14:23:49 2009 EDT

  Modified files:              
    /loncom/auth	lonlogin.pm 
  Log:
  Info Box:
  - Consistent design and indention for server admin e-mail address, version number, new account link
  - Moved line breaks from subroutines which create the links/texts to the block where they are displayed
  - Separate contact block from rest of info fields
  - XHTML: Corrected label references for uname, password, domain
  
  
Index: loncom/auth/lonlogin.pm
diff -u loncom/auth/lonlogin.pm:1.129 loncom/auth/lonlogin.pm:1.130
--- loncom/auth/lonlogin.pm:1.129	Fri Oct 23 16:14:36 2009
+++ loncom/auth/lonlogin.pm	Tue Dec  1 14:23:49 2009
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Login Screen
 #
-# $Id: lonlogin.pm,v 1.129 2009/10/23 16:14:36 bisitz Exp $
+# $Id: lonlogin.pm,v 1.130 2009/12/01 14:23:49 bisitz Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -390,7 +390,7 @@
 }
 my $newuserlink;
 if ($shownewuserlink) {
-    $newuserlink = &newuser_link($lt{'newuser'}).'<br />';
+    $newuserlink = &newuser_link($lt{'newuser'});
 }
 my $logintitle;
 if ($loginheader eq 'text') {
@@ -412,11 +412,11 @@
   <input type="hidden" name="lextkey" value="$lextkey" />
   <input type="hidden" name="uextkey" value="$uextkey" />
   <b><label for="uname">$lt{'un'}</label>:</b><br />
-  <input type="text" name="uname" size="15" value="$authusername" /><br />
+  <input type="text" name="uname" id="uname" size="15" value="$authusername" /><br />
   <b><label for="upass$now">$lt{'pw'}</label>:</b><br />
-  <input type="password" name="upass$now" size="15" /><br />
+  <input type="password" name="upass$now" id="upass$now" size="15" /><br />
   <b><label for="udom">$lt{'dom'}</label>:</b><br />
-  <input type="text" name="udom" size="15" value="$authdomain" /><br />
+  <input type="text" name="udom" id="udom" size="15" value="$authdomain" /><br />
   <input type="submit" value="$lt{'log'}" />
 </form>
 LFORM
@@ -449,11 +449,13 @@
 </div>
   
 <div class="LC_loginpage_loginInfo">
-        $loginhelp<br />
-        $forgotpw<br />
-	$contactblock<br />
-        $newuserlink
-        $coursecatalog
+  $loginhelp<br />
+  $forgotpw<br />
+  <br />
+  $contactblock<br />
+  <br />
+  $newuserlink<br />
+  $coursecatalog
 </div>
 </div>
 ENDTOP
@@ -540,8 +542,8 @@
         $showhelpdesk = 1;
     }
     if ($servadm && $showadminmail) {
-        $contactblock .= '<b>&nbsp;&nbsp;&nbsp;'.$$lt{'servadm'}.':</b><br />'.
-                         '<tt>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;'.$servadm.'</tt><br />&nbsp;<br />';
+        $contactblock .= $$lt{'servadm'}.':<br />'.
+                         '<tt>'.$servadm.'</tt><br />';
     }
     if ($showhelpdesk) {
         $contactblock .= '<a href="javascript:helpdesk()">'.$lt->{'helpdesk'}.'</a><br />';
@@ -562,9 +564,7 @@
 </script>
 ENDSCRIPT
     }
-    $contactblock .= <<"ENDBLOCK";
-     &nbsp;&nbsp;&nbsp;$version
-ENDBLOCK
+    $contactblock .= $version;
     return $contactblock;
 }
 
@@ -595,7 +595,7 @@
 
 sub newuser_link {
     my ($linkname) = @_;
-    return '&nbsp;&nbsp;&nbsp;<a href="/adm/createaccount"><b>'.$linkname.'</b></a><br />';
+    return '<a href="/adm/createaccount">'.$linkname.'</a>';
 }
 
 1;