From 68cb6bab5bba5094ec1a72c34ccd51681dcbe328 Mon Sep 17 00:00:00 2001
From: "Karl O. Pinc" <kop@karlpinc.com>
Date: Sat, 3 Aug 2024 16:15:40 -0500
Subject: [PATCH] Use table row abstractions

---
 src/pgwui_copy/templates/copy.mak | 50 ++++++++++++-------------------
 1 file changed, 19 insertions(+), 31 deletions(-)

diff --git a/src/pgwui_copy/templates/copy.mak b/src/pgwui_copy/templates/copy.mak
index 1b638d0..2e6d622 100644
--- a/src/pgwui_copy/templates/copy.mak
+++ b/src/pgwui_copy/templates/copy.mak
@@ -70,10 +70,8 @@
 
 <%def name="schema_row(tab_index)">
   <tr>
-    <td class="label">
-      <label for="schema_id">Schema To Copy:</label>
-    </td>
-    <td>
+    <%self.lib:td_label for_id="schema_id">Schema To Copy</%self.lib:td_label>
+    <%self.lib:td_input tab_index="${tab_index}">
       <input id="schema_id"
              name="schema"
              tabindex="${tab_index.val}"
@@ -81,17 +79,14 @@
              size="30"
              value="${schema}"
              />
-    </td>
+    </%self.lib:td_input>
   </tr>
-  <% tab_index.inc() %>
 </%def>
 
 <%def name="from_db_row(tab_index)">
   <tr>
-    <td class="label">
-      <label for="from_db_id">From Database:</label>
-    </td>
-    <td>
+    <%self.lib:td_label for_id="from_db_id">From Database</%self.lib:td_label>
+    <%self.lib:td_input tab_index="${tab_index}">
       <input id="from_db_id"
              name="from_db"
              tabindex="${tab_index.val}"
@@ -99,17 +94,14 @@
              size="30"
              value="${from_db}"
              />
-    </td>
+    </%self.lib:td_input>
   </tr>
-  <% tab_index.inc() %>
 </%def>
 
 <%def name="to_db_row(tab_index)">
   <tr>
-    <td class="label">
-      <label for="to_db_id">To Database:</label>
-    </td>
-    <td>
+    <%self.lib:td_label for_id="to_db_id">To Database</%self.lib:td_label>
+    <%self.lib:td_input tab_index="${tab_index}">
       <input id="to_db_id"
              name="to_db"
              tabindex="${tab_index.val}"
@@ -117,44 +109,40 @@
              size="30"
              value="${to_db}"
              />
-    </td>
+    </%self.lib:td_input>
   </tr>
-  <% tab_index.inc() %>
 </%def>
 
 <%def name="overwrite_row(tab_index)">
   <tr>
-    <td class="label">
-      <label for="overwrite_id">Overwrite destination schema<span
-                                style="caution">*</span>:</label>
-    </td>
-    <td>
+    <%self.lib:td_label for_id="overwrite_id">
+      Overwrite destination schema<span style="caution">*</span>
+    </%self.lib:td_label>
+    <%self.lib:td_input tab_index="${tab_index}">
       <input name="overwrite"
              tabindex="${tab_index.val}"
              id="overwrite_id"
              type="checkbox"
              ${overwrite | n}
              />
-    </td>
+    </%self.lib:td_input>
   </tr>
-  <% tab_index.inc() %>
 </%def>
 
 <%def name="force_row(tab_index)">
   <tr>
-    <td class="label">
-      <label for="force_id">Allow Copy To Sensitive DBs:</label>
-      </td>
-    <td>
+    <%self.lib:td_label for_id="force_id">
+      Allow Copy To Sensitive DBs
+    </%self.lib:td_label>
+    <%self.lib:td_input tab_index="${tab_index}">
       <input name="force"
              tabindex="${tab_index.val}"
              id="force_id"
              type="checkbox"
              ${force | n}
               />
-    </td>
+    </%self.lib:td_input>
   </tr>
-  <% tab_index.inc() %>
 </%def>
 
 <%block name="page_heading">
-- 
2.34.1