Bug 1200765 - Make login UX mobile friendly to assist mobile authentication workflow

3 files changed, 67 insertions, 60 deletions

diff --git a/template/en/default/account/auth/login.html.tmpl b/template/en/default/account/auth/login.html.tmpl
index 922a55bd4..33aeaeaea 100644
--- a/template/en/default/account/auth/login.html.tmpl
+++ b/template/en/default/account/auth/login.html.tmpl
@@ -30,8 +30,9 @@
 [% PROCESS global/variables.none.tmpl %]
 
 [% PROCESS global/header.html.tmpl
-  title = "Log in to $terms.Bugzilla",
-  onload = "document.forms['login'].Bugzilla_login.focus()"
+  title        = "Log in to $terms.Bugzilla",
+  onload       = "document.forms['login'].Bugzilla_login.focus()"
+  allow_mobile = 1
 %]
 
 [% USE Bugzilla %]
@@ -40,59 +41,55 @@
   I need an email address and password to continue.
 </p>
 
-<form name="login" action="[% target FILTER html %]" method="POST"
-[%- IF Bugzilla.cgi.param("data") %] enctype="multipart/form-data"[% END %]>
-  <table>
-    <tr>
-      <th align="right"><label for="Bugzilla_login">Email Address:</label></th>
-      <td>
-        <input size="35" id="Bugzilla_login" name="Bugzilla_login"
-               [%- ' type="email"' UNLESS Param('emailsuffix') %]>
-        [% Param('emailsuffix') FILTER html %]
-      </td>
-    </tr>
-    <tr>
-      <th align="right"><label for="Bugzilla_password">Password:</label></th>
-      <td>
-        <input type="password" size="35" id="Bugzilla_password" name="Bugzilla_password">
-      </td>
-    </tr>
+<div id="login" class="login-form">
+  <form name="login" action="[% target FILTER html %]" method="POST"
+        [%- IF Bugzilla.cgi.param("data") %] enctype="multipart/form-data"[% END %]>
+    <div class="field-login">
+      <label for="Bugzilla_login">Email Address:</label>
+      <input id="Bugzilla_login" name="Bugzilla_login"
+             [%- ' type="email"' UNLESS Param('emailsuffix') %]>
+      [% Param('emailsuffix') FILTER html %]
+    </div>
+
+    <div class="field-password">
+      <label for="Bugzilla_password">Password:</label>
+      <input type="password" id="Bugzilla_password" name="Bugzilla_password">
+    </div>
 
     [% IF Param('rememberlogin') == 'defaulton' || 
-          Param('rememberlogin') == 'defaultoff' %]
-      <tr>
-        <th>&nbsp;</th>
-        <td>
-          <input type="checkbox" id="Bugzilla_remember" name="Bugzilla_remember" value="on"
-                 [%+ "checked" IF Param('rememberlogin') == "defaulton" %]>
-          <label for="Bugzilla_remember">Remember my email address</label>
-        </td>
-      </tr>
+       Param('rememberlogin') == 'defaultoff' %]
+      <div class="field-remember">
+        <input type="checkbox" id="Bugzilla_remember" name="Bugzilla_remember" value="on"
+               [%+ "checked" IF Param('rememberlogin') == "defaulton" %]>
+        <label for="Bugzilla_remember" class="checkbox-note">
+          Remember my email address
+        </label>
+      </div>
     [% END %]
 
-    <tr>
-      <th>&nbsp;</th>
-      <td>
-        <input type="checkbox" id="Bugzilla_restrictlogin" name="Bugzilla_restrictlogin"
-               checked="checked">
-        <label for="Bugzilla_restrictlogin">Restrict this session to this IP address
-        (using this option improves security)</label>
-      </td>
-    </tr>
-  </table>
+    [% PROCESS "global/hidden-fields.html.tmpl"
+       exclude="^Bugzilla_(login|password|restrictlogin)$" %]
 
-  [% PROCESS "global/hidden-fields.html.tmpl"
-     exclude="^Bugzilla_(login|password|restrictlogin)$" %]
+    <div class="field-restrict">
+      <input type="checkbox" id="Bugzilla_restrictlogin" name="Bugzilla_restrictlogin"
+             checked="checked">
+      <label for="Bugzilla_restrictlogin" class="checkbox-note">
+        Restrict this session to this IP address
+        (using this option improves security)</label>
+    </div>
 
-  <input type="hidden" name="Bugzilla_login_token"
-         value="[% get_login_request_token() FILTER html %]">
-  <input type="submit" name="GoAheadAndLogIn" value="Log in" id="log_in">
+    <div class="field-submit">
+      <input type="hidden" name="Bugzilla_login_token"
+             value="[% get_login_request_token() FILTER html %]">
+      <input type="submit" name="GoAheadAndLogIn" value="Log in" id="log_in">
+    </div>
 
-  <p>
-    (Note: you should make sure cookies are enabled for this site. 
-    Otherwise, you will be required to log in frequently.)
-  </p>
-</form>
+    <p class="cookie-notify">
+      (Note: you should make sure cookies are enabled for this site.
+      Otherwise, you will be required to log in frequently.)
+    </p>
+  </form>
+</div>
 
 [% Hook.process('additional_methods') %]
 
@@ -117,7 +114,7 @@
       If you have an account, but have forgotten your password,
       enter your email address below and submit a request
       to change your password.<br>
-      <input size="35" name="loginname">
+      <input name="loginname">
       <input type="hidden" id="token" name="token" value="[% issue_hash_token(['reqpw']) FILTER html %]">
       <input type="submit" id="request" value="Reset Password">
     </form>
diff --git a/template/en/default/global/header.html.tmpl b/template/en/default/global/header.html.tmpl
index 3f70b9453..23634ed43 100644
--- a/template/en/default/global/header.html.tmpl
+++ b/template/en/default/global/header.html.tmpl
@@ -37,6 +37,7 @@
   # atomlink: Atom link URL, May contain HTML
   # generate_api_token: generate a token which can be used to make authenticated webservice calls
   # no_body: if true the body element will not be generated
+  # allow_mobile: allow special CSS and viewport for detected mobile useragents
   #%]
 
 [% IF message %]
@@ -103,6 +104,10 @@
 [%# Add our required jQuery plugins %]
 [% jquery.push("cookie", "devbridgeAutocomplete") %]
 
+[% IF allow_mobile && is_mobile_browser %]
+  [% style_urls.push("skins/standard/mobile.css") %]
+[% END %]
+
 [%# We should be able to set the default value of the header variable
   # to the value of the title variable using the DEFAULT directive,
   # but that doesn't work if a caller sets header to the empty string
@@ -260,6 +265,9 @@
     [%# Required for the 'Autodiscovery' feature in Firefox 2 and IE 7. %]
     <link rel="search" type="application/opensearchdescription+xml"
                        title="[% terms.BugzillaTitle %]" href="./search_plugin.cgi">
+    [% IF allow_mobile && is_mobile_browser %]
+      <meta name="viewport" content="width=device-width, initial-scale=1">
+    [% END %]
     [% Hook.process("additional_header") %]
   </head>
 
diff --git a/template/en/default/mfa/totp/verify.html.tmpl b/template/en/default/mfa/totp/verify.html.tmpl
index ad75dc6bc..7314d909a 100644
--- a/template/en/default/mfa/totp/verify.html.tmpl
+++ b/template/en/default/mfa/totp/verify.html.tmpl
@@ -8,6 +8,7 @@
 
 [% INCLUDE global/header.html.tmpl
   title = "Account Verification"
+  allow_mobile = 1
 %]
 
 <h1>Account Verification</h1>
@@ -16,16 +17,17 @@
   <b>[% reason FILTER html %]</b> requires verification.<br>
   Please enter your verification code from your TOTP application:
 </p>
-
-<form method="POST" action="[% postback.action FILTER none %]">
-  [% FOREACH field IN postback.fields.keys %]
-    <input type="hidden" name="[% field FILTER html %]" value="[% postback.fields.item(field) FILTER html %]">
-  [% END %]
-  <input type="text" name="code" id="code"
-        placeholder="123456" maxlength="9" pattern="\d{6,9}" size="10"
-        autocomplete="off" required autofocus><br>
-  <br>
-  <input type="submit" value="Submit">
-</form>
+<div class="verify-totp">
+  <form method="POST" action="[% postback.action FILTER none %]">
+    [% FOREACH field IN postback.fields.keys %]
+      <input type="hidden" name="[% field FILTER html %]" value="[% postback.fields.item(field) FILTER html %]">
+    [% END %]
+    <input type="text" name="code" id="code"
+           placeholder="123456" maxlength="9" pattern="\d{6,9}" size="10"
+           autocomplete="off" required autofocus><br>
+    <br>
+    <input type="submit" value="Submit">
+  </form>
+</div>
 
 [% INCLUDE global/footer.html.tmpl %]