From: Pasi Kallinen <pasi.kallinen@pttk.fi>
Date: Thu, 31 Jan 2013 10:20:48 +0000 (+0200)
Subject: Clean up the account summary table HTML.
X-Git-Tag: sprint4-merge-nov22~2979
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=15d7de9d1f184de84c00ae40128663e11f972f7c;p=working%2FEvergreen.git

Clean up the account summary table HTML.

Instead of faking a table with three separate DIVs,
actually use a single table.

Signed-off-by: Pasi Kallinen <pasi.kallinen@pttk.fi>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
Signed-off-by: Dan Scott <dscott@laurentian.ca>
---

diff --git a/Open-ILS/src/templates/opac/css/style.css.tt2 b/Open-ILS/src/templates/opac/css/style.css.tt2
index 06897f6b43..f73aaa2076 100644
--- a/Open-ILS/src/templates/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates/opac/css/style.css.tt2
@@ -789,22 +789,25 @@ div.result_place_hold {
     font-weight: normal;
 }
 
-.acct_sum_row {
-    padding: 7px 15px;
+.acct_sum_table {
+    border-collapse: collapse;
     background: [% css_colors.accent_ultralight %];
-    margin-bottom: 2px;
     font-size: [% css_fonts.size_smaller %];
     font-weight: bold;
     text-transform: uppercase;
 }
 
-.acct_sum_row a {
+.acct_sum_table td {
+    padding: 1em;
+}
+
+.acct_sum_table a {
     text-transform: none;
     position:relative;
     top:-1px;
 }
 
-.acct_sum_row .view_link {
+.acct_sum_table .view_link {
     font-weight: normal;
 }
 
diff --git a/Open-ILS/src/templates/opac/parts/myopac/main_base.tt2 b/Open-ILS/src/templates/opac/parts/myopac/main_base.tt2
index f0685e097b..b14572e02f 100644
--- a/Open-ILS/src/templates/opac/parts/myopac/main_base.tt2
+++ b/Open-ILS/src/templates/opac/parts/myopac/main_base.tt2
@@ -57,37 +57,26 @@
             <span class="alert">[% l("Your library card expired on [_1]. Please contact a librarian to resolve this issue.", fmt_expire_date) %]</span>
             [% END %]
             </div>
-            <div class="acct_sum_row">
-                <table width="100%" cellspacing="0" cellpadding="0">
-                    <tr>
+            <table class="acct_sum_table">
+                <tr>
                         <td>[% l("Items Currently Checked out ([_1])", ctx.user_stats.checkouts.total_out) %]</td>
                         <td align="right">
                             <a href="[% mkurl(ctx.opac_root _ '/myopac/circs') %]">[% l("View All") %]</a>
                         </td>
-                    </tr>
-                </table>
-            </div>
-            <div class="acct_sum_row">
-                <table width="100%" cellspacing="0" cellpadding="0">
-                    <tr>
+                </tr>
+                <tr>
                         <td>[% l('Items Currently on Hold ([_1])', ctx.user_stats.holds.total) %]</td>
                         <td align="right">
                             <a href="[% mkurl(ctx.opac_root _ '/myopac/holds') %]">[% l('View All') %]</a>
                         </td>
-                    </tr>
-                </table>
-            </div>
-            <div class="acct_sum_row">
-                <table width="100%" cellspacing="0" cellpadding="0">
-                    <tr>
+                </tr>
+                <tr>
                         <td>[% l('Items ready for pickup ([_1])', ctx.user_stats.holds.ready) %]</td>
                         <td align="right">
                             <a href="[% mkurl(ctx.opac_root _ '/myopac/holds', {available => 1}) %]">[% l('View All') %]</a>
                         </td>
-                    </tr>
-                </table>
-            </div>
-
+                </tr>
+            </table>
         </div>
     </div>
     <div class="clear-both"></div>