Make username/password required in login form

On login form, require both password and login name. Neither can be
empty.  Addeing required to both input field will invoke browser
validation (where available) indicating a problem when logging in.

Author: rouilj

share/roundup/templates/classic/html/page.html

@@ -129,8 +129,8 @@ <h2><span metal:define-slot="body_title">body title</span></h2>
         tal:attributes="action request/base">
    <p class="userblock">
     <b i18n:translate="">Login</b><br>
-    <input size="10" name="__login_name"><br>
-    <input size="10" type="password" name="__login_password"><br>
+    <input size="10" required name="__login_name"><br>
+    <input size="10" type="password" required name="__login_password"><br>
     <input type="hidden" name="@action" value="Login">
     <input type="checkbox" name="remember" id="remember">
     <label for="remember" i18n:translate="">Remember me?</label><br>

share/roundup/templates/devel/html/page.html

@@ -162,8 +162,8 @@ <h1><a href="/">Roundup Demo Tracker</a></h1>
        <ul>
         <li>
          <tal:span i18n:translate="">Login</tal:span><br/>
-         <input size="10" name="__login_name"/><br/>
-         <input size="10" type="password" name="__login_password"/><br/>
+         <input size="10" required name="__login_name"/><br/>
+         <input size="10" type="password" required name="__login_password"/><br/>
 	 <input name="@csrf" type="hidden"
 		tal:attributes="value python:utils.anti_csrf_nonce()">
          <input type="hidden" name="@action" value="Login"/>

share/roundup/templates/jinja2/html/layout/navigation.html

@@ -110,10 +110,10 @@
         <i class='icon-user'></i>{% trans %}Login{% endtrans %}
       </li>
       <li class="nav-item">
-        <input class="form-control form-control-sm" type='text' name="__login_name" placeholder='username'>
+        <input class="form-control form-control-sm" type='text' required name="__login_name" placeholder='username'>
       </li>
       <li class="nav-item">
-        <input class="form-control form-control-sm" type="password" name="__login_password" placeholder='password'>
+        <input class="form-control form-control-sm" type="password" required name="__login_password" placeholder='password'>
       </li>
       <li class="nav-item">
         <label class="form-control form-control-sm" class='checkbox'>

share/roundup/templates/minimal/html/page.html

@@ -129,8 +129,8 @@ <h2><span metal:define-slot="body_title">body title</span></h2>
         tal:attributes="action request/base">
    <p class="userblock">
     <b i18n:translate="">Login</b><br>
-    <input size="10" name="__login_name"><br>
-    <input size="10" type="password" name="__login_password"><br>
+    <input size="10" required name="__login_name"><br>
+    <input size="10" type="password" required name="__login_password"><br>
     <input name="@csrf" type="hidden"
            tal:attributes="value python:utils.anti_csrf_nonce()">
     <input type="hidden" name="@action" value="Login">

share/roundup/templates/responsive/html/page.html

@@ -177,8 +177,8 @@
               <form method="post" tal:attributes="action request/base">
                 <fieldset>
                   <legend><i class='icon-user'></i>Login form</legend>
-                  <input name="__login_name" type='text' placeholder='Username' i18n:attributes="placeholder">
-                  <input type="password" name="__login_password" placeholder='Password' i18n:attributes="placeholder">
+                  <input name="__login_name" type='text' placeholder='Username' i18n:attributes="placeholder" required>
+                  <input type="password" name="__login_password" placeholder='Password' i18n:attributes="placeholder" required>
 		  <input name="@csrf" type="hidden"
 			 tal:attributes="value python:utils.anti_csrf_nonce()">
                   <input type="hidden" name="@action" value="Login"/>