From 5718324ede7eb1b7ab7a9c353ae6b8e79788f0a8 Mon Sep 17 00:00:00 2001
From: Dan Scott <dscott@laurentian.ca>
Date: Mon, 12 Sep 2011 16:02:45 -0400
Subject: [PATCH] CSSize fine/payment tabs on My Account page

More consolidation of CSS is a win for making changes in one place.
More accessibility as well.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
---
 Open-ILS/src/templates/opac/parts/myopac/base.tt2  |  2 +-
 .../src/templates/opac/parts/myopac/main_base.tt2  |  8 +++---
 Open-ILS/web/css/skin/default/opac/style.css       | 32 ++++++----------------
 3 files changed, 13 insertions(+), 29 deletions(-)

diff --git a/Open-ILS/src/templates/opac/parts/myopac/base.tt2 b/Open-ILS/src/templates/opac/parts/myopac/base.tt2
index d0d79ddf50..62291cfbd4 100644
--- a/Open-ILS/src/templates/opac/parts/myopac/base.tt2
+++ b/Open-ILS/src/templates/opac/parts/myopac/base.tt2
@@ -25,7 +25,7 @@
                         cls_which = "acct-tab-off";
                     END -%]
                 <a href="[% ctx.opac_root _ '/myopac/' _ page.url %]"
-                    class="[% cls_which %] acct-tab">[% page.name; %]</a>
+                    class="[% cls_which %]">[% page.name; %]</a>
                 [% END %]
             </div>
         </div>
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 62e317424c..850673aea2 100644
--- a/Open-ILS/src/templates/opac/parts/myopac/main_base.tt2
+++ b/Open-ILS/src/templates/opac/parts/myopac/main_base.tt2
@@ -86,11 +86,11 @@
     <div id='fines_payments_wrapper'>
         <div id='acct_fines_tabs'>
             [% IF myopac_main_page == 'main' %]
-            <a href='#'><img src='[% ctx.media_prefix %]/images/acct_fines_on.jpg'/></a>
-            <a href='main_payments'><img src='[% ctx.media_prefix %]/images/acct_payments_off.jpg'/></a>
+            <a href='#' class="acct-tab-on">[% l("Fines") %]</a>
+            <a href='main_payments' class="acct-tab-off">[% l("Payments") %]</a>
             [% ELSE %]
-            <a href='main'><img src='[% ctx.media_prefix %]/images/acct_fines_off.jpg'/></a>
-            <a href='#'><img src='[% ctx.media_prefix %]/images/acct_payments_on.jpg'/></a>
+            <a href='main' class="acct-tab-off">[% l("Fines") %]</a>
+            <a href='#' class="acct-tab-on">[% l("Payments") %]</a>
             [% END %]
         </div>
     </div>
diff --git a/Open-ILS/web/css/skin/default/opac/style.css b/Open-ILS/web/css/skin/default/opac/style.css
index 29227be0e0..b6d618005d 100644
--- a/Open-ILS/web/css/skin/default/opac/style.css
+++ b/Open-ILS/web/css/skin/default/opac/style.css
@@ -273,13 +273,13 @@ div.select-wrapper:hover {
 	color: white;
 	font-size: 10px;
 }
-#adv_search_tabs {
+#adv_search_tabs, #acct_tabs, #acct_fines_tabs {
 	height: 40px;
 	width:974px;
 	margin:auto;
 }
 
-#adv_search_tabs a, #acct_tabs a {
+#adv_search_tabs a, #acct_tabs a, #acct_fines_tabs a {
     float: left;
 	text-align: center;
     vertical-align: middle;
@@ -288,41 +288,25 @@ div.select-wrapper:hover {
     padding: 10px 0px 10px 0px;
     -moz-border-radius: 5%; 
     border-radius: 5%;
-}
-
-#adv_search, #num_search, #expert_search {
+    font-weight: bold;
 	width:156px;
     color: 45709b;
     background: #9ad0f1;
     font-weight: bold;
 }
+
 #adv_search.on, #num_search.on, #expert_search.on {
     color: #333333;
     background: white;
 }
 
-#acct_tabs, #acct_fines_tabs {
-	height:33px;
-	width:974px;
-	margin:auto;
-}
-
-#acct_fines_tabs a {
-	float: left;
-	display: block;
-	height:33px;
-	margin-right:7px;
+#adv_search_tabs a.acct-tab-on, #acct_tabs a.acct-tab-on, #acct_fines_tabs a.acct-tab-on {
+    color: #333333;
+    background: white;
 }
 
-.acct-tab {
-    width:156px;
+.acct-tab-off {
     background: #9ad0f1;
-    font-weight: bold;
-}
-
-.acct-tab-on {
-    color: #333333;
-    background: white;
 }
 
 #rdetail_header {
-- 
2.11.0