From: Karl O. Pinc <kop@karlpinc.com>
Date: Sun, 29 Sep 2024 21:03:48 +0000 (-0500)
Subject: Do not use a for= id on a label without an input element
X-Git-Url: https://papio.biology.duke.edu/gitweb/?a=commitdiff_plain;h=1240e28284dfe20b0b387fdbb8e801b768eb9056;p=pgwui_common

Do not use a for= id on a label without an input element
---

diff --git a/src/pgwui_common/templates/auth_base.mak b/src/pgwui_common/templates/auth_base.mak
index 7217d0b..35c87ed 100644
--- a/src/pgwui_common/templates/auth_base.mak
+++ b/src/pgwui_common/templates/auth_base.mak
@@ -97,22 +97,23 @@
 
 <%def name="user_row(tab_index, havecreds, user)">
       <tr>
-        <%self.lib:td_label for_id="user_id">Username</%self.lib:td_label>
-          % if havecreds:
-            <%self.lib:td_input tab_index="${tab_index}" tab_inc="${0}">
-              <span id="user_id">${user}</span>
-            </%self.lib:td_input>
-          % else:
-            <%self.lib:td_input tab_index="${tab_index}">
-              <input name="user"
-                     tabindex="${tab_index.val}"
-                     id="user_id"
-                     type="text"
-                     size="30"
-                     value=""
-                     />
-            </%self.lib:td_input>
-          % endif
+        % if havecreds:
+          <%self.lib:td_label>Username</%self.lib:td_label>
+          <%self.lib:td_input tab_index="${tab_index}" tab_inc="${0}">
+            <span id="user_id">${user}</span>
+          </%self.lib:td_input>
+        % else:
+          <%self.lib:td_label for_id="user_id">Username</%self.lib:td_label>
+          <%self.lib:td_input tab_index="${tab_index}">
+            <input name="user"
+                   tabindex="${tab_index.val}"
+                   id="user_id"
+                   type="text"
+                   size="30"
+                   value=""
+                   />
+          </%self.lib:td_input>
+        % endif
       </tr>
 </%def>