From 74355312a6df553cc1b83ad3095821cf39b28910 Mon Sep 17 00:00:00 2001
From: berick <berick@esilibrary.com>
Date: Fri, 18 Feb 2011 13:12:50 -0500
Subject: [PATCH] round of CSS cleanup;  removing unused rules and replacing
 inline style with css classes

---
 Open-ILS/web/css/skin/default/opac/semiauto.css    | 15 ++++------
 Open-ILS/web/css/skin/default/opac/style.css       |  2 +-
 .../web/templates/default/opac/myopac/circs.tt2    | 32 ++++++++--------------
 .../web/templates/default/opac/myopac/main.tt2     |  6 ++--
 .../web/templates/default/opac/myopac/prefs.tt2    |  5 ++--
 .../default/opac/parts/advanced/search.tt2         | 10 +++----
 .../templates/default/opac/parts/place_hold.tt2    |  3 +-
 .../default/opac/parts/record/cn_details.tt2       |  4 +--
 .../default/opac/parts/record/copyinfo.tt2         |  2 +-
 .../templates/default/opac/parts/record/extras.tt2 |  2 +-
 .../default/opac/parts/record/summary.tt2          |  7 ++---
 .../templates/default/opac/parts/result/table.tt2  | 13 ++++-----
 12 files changed, 41 insertions(+), 60 deletions(-)

diff --git a/Open-ILS/web/css/skin/default/opac/semiauto.css b/Open-ILS/web/css/skin/default/opac/semiauto.css
index 6902837cfa..550669f04c 100644
--- a/Open-ILS/web/css/skin/default/opac/semiauto.css
+++ b/Open-ILS/web/css/skin/default/opac/semiauto.css
@@ -1,23 +1,15 @@
 /* once done renaming and everything, combine with style.css */
-.opac-auto-001 { background: #2bb0fd; }
-.opac-auto-002 { background: #54987f; }
-.opac-auto-003 { background: #decc92; }
+
+/* Verfied in-use classes --------------------------- */
 .opac-auto-004 { background: #E0F0E0; }
-.opac-auto-005 { background: #ffc331; }
-.gray-back { background: gray; }
-.pink-back { background: pink; }
-.white-back { background: white; }
 .opac-auto-009 { border: 3px solid #E0E0E0; }
 .opac-auto-010 { border-bottom: 1px dotted #ccc; padding-top: 10px; }
 .opac-auto-011 { border-bottom: 1px dotted #ccc; padding-top: 6px; }
 .opac-auto-012 { border-bottom: none; }
 .opac-auto-013 { border-bottom: none; *height: 0px; }
-.opac-auto-014 { border-collapse: collapse; }
 .opac-auto-015 { border-left: 1px solid #e9ebf3; padding-right: 27px; }
-.opac-auto-016 { border: none; }
 .opac-auto-017 { border: none; width: 100%; }
 .opac-auto-018 { border-top: 1px dotted #ccc; padding-top: 17px; }
-.opac-auto-019 { border-top: none; }
 .clear-both { clear: both; }
 .common-no-pad { clear: both; height: 0px; margin: 0px; padding: 0px; }
 .common-full-pad { clear: both; height: 15px; }
@@ -27,6 +19,9 @@
 .red { color: red; }
 .bold-red { color: red; font-weight: bold }
 .pointer { cursor: pointer; }
+/* ------------------------------------------- */
+
+
 #lib_selector_span { display: none }
 .float-left { float: left; }
 .opac-auto-039 { float: left; font-weight: bold; padding-top: 5px; }
diff --git a/Open-ILS/web/css/skin/default/opac/style.css b/Open-ILS/web/css/skin/default/opac/style.css
index 3a8a94aee7..e47edcc671 100644
--- a/Open-ILS/web/css/skin/default/opac/style.css
+++ b/Open-ILS/web/css/skin/default/opac/style.css
@@ -58,7 +58,7 @@ h2 {
 	font-weight:bold;
 }
 
-.hide_me {
+.hide_me, .hidden {
 	display: none;
 	visibility: hidden;
 }
diff --git a/Open-ILS/web/templates/default/opac/myopac/circs.tt2 b/Open-ILS/web/templates/default/opac/myopac/circs.tt2
index 2e1d3a4442..ea7842661d 100644
--- a/Open-ILS/web/templates/default/opac/myopac/circs.tt2
+++ b/Open-ILS/web/templates/default/opac/myopac/circs.tt2
@@ -70,27 +70,25 @@
                     />
                 </td>
                 <td width="40%" style="padding-left:5px;">
-                    <span title="Click to sort" style="cursor:pointer;">
+                    <span title="Click to sort" class='pointer'>
                         Title
                     </span> /
-                    <span title="Click to sort" style="cursor:pointer;">
+                    <span title="Click to sort" class='pointer'>
                         Author
                     </span>
                 </td>
                 <td width="8%" style="padding-right:5px;" align="center">
-                    <span title="Click to sort" style="cursor:pointer;">
-                        Renews<br />Left
+                    <span title="Click to sort" class='pointer'>Renews<br />Left
                     </span>
                 </td>
                 <td width="13%" style="padding-left:5px;">
-                    <span title="Click to sort" style="cursor:pointer;">Due Date
-                    </span>
+                    <span title="Click to sort" class='pointer'>Due Date</span>
                 </td>
                 <td width="16%">
-                    <span title="Click to sort" style="cursor:pointer;">barcode</span>
+                    <span title="Click to sort" class='pointer'>barcode</span>
                 </td>
                 <td width="22%">
-                    <span title="Click to sort" style="cursor:pointer;">call number</span>
+                    <span title="Click to sort" class='pointer'>call number</span>
                 </td>
             </tr>
         </table>
@@ -151,27 +149,21 @@
             border='0' width='100%'>
             <tr>
                 <td width="45%" style="padding-left:15px;">
-                    <span title="Click to sort"
-                        style="cursor:pointer;">Title</span> /
-                    <span title="Click to sort"
-                        style="cursor:pointer;">Author</span>
+                    <span title="Click to sort" class='pointer'>Title</span> /
+                    <span title="Click to sort" class='pointer'>Author</span>
                 </td>
                 <td width="22%"
                     style="white-space:nowrap;padding-left:5px;">
-                    <span title="Click to sort"
-                        style="cursor:pointer;">Call Number</span>
+                    <span title="Click to sort" class='pointer'>Call Number</span>
                 </td>
                 <td width="11%">
-                    <span title="Click to sort"
-                        style="cursor:pointer;">Checkout</span>
+                    <span title="Click to sort" class='pointer'>Checkout</span>
                 </td>
                 <td width="11%">
-                    <span title="Click to sort"
-                        style="cursor:pointer;">Due Date</span>
+                    <span title="Click to sort" class='pointer'>Due Date</span>
                 </td>
                 <td width="11%">
-                    <span title="Click to sort"
-                        style="cursor:pointer;">Returned</span>
+                    <span title="Click to sort" class='pointer'>Returned</span>
                 </td>
             </tr>
         </table>
diff --git a/Open-ILS/web/templates/default/opac/myopac/main.tt2 b/Open-ILS/web/templates/default/opac/myopac/main.tt2
index 2d7b375092..694185a290 100644
--- a/Open-ILS/web/templates/default/opac/myopac/main.tt2
+++ b/Open-ILS/web/templates/default/opac/myopac/main.tt2
@@ -121,14 +121,14 @@
                     <tr id='myopac_fines_summary_row'>
                         <td id='myopac_fines_summary_total'>[% money(ctx.fines.total_owed) %]</td>
                         <td id='myopac_fines_summary_paid'>[% money(ctx.fines.total_paid) %]</td>
-                        <td id='myopac_fines_summary_balance' style='color:red;font-weight: bold;'>[% money(ctx.fines.balance_owed) %]</td>
+                        <td id='myopac_fines_summary_balance' class='bold-red'>[% money(ctx.fines.balance_owed) %]</td>
                     </tr>
                 </tbody>
             </table>
         <!--
         <div id='accrue_explanation' class='hide_me'>
             <span>Transactions whose balances are marked with a</span>
-            <span style='color:red; font-weight: bold'>*</span>
+            <span class='bold-red'>*</span>
             <span>will continue to accrue fines until the checked out item is returned.</span>
         </div>
         -->
@@ -220,7 +220,7 @@
             <!-- Table for all non-circulation transactions -->
             <div id='myopac_trans_div'>
                 <br/>
-                <hr style="border-bottom:none;*height:0px;" color="#dcdbdb" />
+                <hr class='opac-auto-013'  color="#dcdbdb" />
                 <br/>
                 <table width='100%' class='data_grid data_grid_center'
                     id='myopac_trans_table'>
diff --git a/Open-ILS/web/templates/default/opac/myopac/prefs.tt2 b/Open-ILS/web/templates/default/opac/myopac/prefs.tt2
index 0545872482..9f6177a344 100644
--- a/Open-ILS/web/templates/default/opac/myopac/prefs.tt2
+++ b/Open-ILS/web/templates/default/opac/myopac/prefs.tt2
@@ -249,8 +249,7 @@
                        width='100%'>
                     <tbody>
                         <tr>
-                            <td width='100%'
-                                style='color:red;'>
+                            <td width='100%' class='red'>
                                 [% l("Your account expired on") %]
                                 <span id='myopac.expired.date'></span>!
                                 [% l("Please see a librarian to renew your account.") %]</td>
@@ -585,7 +584,7 @@
                     </tr>
                 </tbody>
             </table><br />
-            <hr style="border-bottom:none;*height:0px;" color="#DCDBDB" />
+            <hr class='opac-auto-013' color="#DCDBDB" />
             <br />
 
             <table width='100%' class='light_border data_grid'>
diff --git a/Open-ILS/web/templates/default/opac/parts/advanced/search.tt2 b/Open-ILS/web/templates/default/opac/parts/advanced/search.tt2
index 833d7f3359..c4fde3ada6 100644
--- a/Open-ILS/web/templates/default/opac/parts/advanced/search.tt2
+++ b/Open-ILS/web/templates/default/opac/parts/advanced/search.tt2
@@ -1,7 +1,7 @@
 <table id='adv_global_search' class='data_grid data_grid_center' width='100%'>
     <tr style='border-bottom: none;'>
         <!-- Contains the user-addable(?) rows to define search class, containment and text -->
-        <td valign='top' style='border-bottom: none;'>
+        <td valign='top' class='opac-auto-012'>
             <table width='100%' id='adv_global_input_table'>
                 <thead>
                     <tr>
@@ -66,7 +66,7 @@
             </td>
             <td valign='top'>
             <strong>[% l("Sort Criteria") %]</strong>
-              <table style='border: none; width: 100%;'>
+              <table class='opac-auto-017'>
                 <tr>
                     <td align='center' width='100%'>
                         <table class='' width='100%'>
@@ -202,7 +202,7 @@
     <tr class='border_4_2'>
         <td align="left" colspan='2'>
             <!-- XXX TODO make a real form, and make this a real submitter -->
-        <img src="[% ctx.media_prefix %]/images/search_btn.gif" alt="[% l('Search') %]"  style="cursor:pointer;" />
+        <img src="[% ctx.media_prefix %]/images/search_btn.gif" alt="[% l('Search') %]"  class='pointer' />
         &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
         <!-- XXX TODO make a real form, and make this a real resetter -->
         <a href="javascript:;" style="position: relative; top: -9px;">Reset Form</a>
@@ -235,7 +235,7 @@
             <!-- XXX TODO make a real form, and make a real submitter (quick
             submit, FKA advGenericSearch() -->
             <img src="[% ctx.media_prefix %]/images/search_btn.gif"
-                alt="[% l('Search') %]" id="adv_quick_submit" style="cursor:pointer;" />
+                alt="[% l('Search') %]" id="adv_quick_submit" class='pointer' />
         </div>
     </div>
 </div>
@@ -269,6 +269,6 @@
             class='classic_link' href='javascript:advAddMARC();'>[% l("Add Row") %]</a>
         <!-- XXX TODO make a real form, and make a real submitter (FKA
         advMARCRun()) -->
-        <img alt="Search" src="[% ctx.media_prefix %]/images/search_btn.gif" style="cursor:pointer;" />
+        <img alt="Search" src="[% ctx.media_prefix %]/images/search_btn.gif" class='pointer' />
     </div>
 </div>
diff --git a/Open-ILS/web/templates/default/opac/parts/place_hold.tt2 b/Open-ILS/web/templates/default/opac/parts/place_hold.tt2
index efe30926de..84d8e715a9 100644
--- a/Open-ILS/web/templates/default/opac/parts/place_hold.tt2
+++ b/Open-ILS/web/templates/default/opac/parts/place_hold.tt2
@@ -238,8 +238,7 @@
                     <td class='holds_cell'>
                         <div style='margin-bottom: 5px;'>
                             <span>[% l("Acceptable Alternative Formats:") %] </span>
-                            <span><a class='classic_link' style='color:red;'
-                                    href='#'>[% l("(Help)") %]</a></span>
+                            <span><a class='classic_link red' href='#'>[% l("(Help)") %]</a></span>
                         </div>
                         <div>[% l("(control-click to select multiple formats)") %]</div>
                     </td>
diff --git a/Open-ILS/web/templates/default/opac/parts/record/cn_details.tt2 b/Open-ILS/web/templates/default/opac/parts/record/cn_details.tt2
index 1156a5a32b..2865026a61 100644
--- a/Open-ILS/web/templates/default/opac/parts/record/cn_details.tt2
+++ b/Open-ILS/web/templates/default/opac/parts/record/cn_details.tt2
@@ -1,6 +1,6 @@
 <table class="hide_me">
     <tbody>
-        <tr style='border: 3px solid #E0E0E0;' id='rdetail_volume_details_row' templateRow='1'>
+        <tr class='opac-auto-009' id='rdetail_volume_details_row' templateRow='1'>
             <td colspan='10'>
                 <div style='text-align: center; margin-top: 6px; margin-bottom: 6px'>
                     <a name='print' href='#' class='classic_link'>[% l("print these details") %]</a>
@@ -78,7 +78,7 @@
                     <tr><td>[% l("Publisher") %]</td><td colspan='2' name='publisher'></td></tr>
                     <tr><td>[% l("Physical Description") %]</td><td colspan='2' name='phys'></td></tr>
                     <tr>
-                        <td colspan='3' style='background: #E0F0E0;'> </td>
+                        <td colspan='3' class='opac-auto-004'> </td>
                     </tr>
                     <tr name='cnrow'>
                         <td><b>[% l("Call Number") %]</b></td>
diff --git a/Open-ILS/web/templates/default/opac/parts/record/copyinfo.tt2 b/Open-ILS/web/templates/default/opac/parts/record/copyinfo.tt2
index 8439eda20c..9d4673f120 100644
--- a/Open-ILS/web/templates/default/opac/parts/record/copyinfo.tt2
+++ b/Open-ILS/web/templates/default/opac/parts/record/copyinfo.tt2
@@ -59,7 +59,7 @@
     </table>
     <!-- <table class='hide_me'>
         <tbody>
-        <tr style='border: 3px solid #E0E0E0;' id='rdetail_volume_details_row'><td colspan='10'></td></tr>
+        <tr class='opac-auto-009' id='rdetail_volume_details_row'><td colspan='10'></td></tr>
         </tbody>
     </table> -->
     <br/><br/>
diff --git a/Open-ILS/web/templates/default/opac/parts/record/extras.tt2 b/Open-ILS/web/templates/default/opac/parts/record/extras.tt2
index 5d714bc2bb..6db9b6e478 100644
--- a/Open-ILS/web/templates/default/opac/parts/record/extras.tt2
+++ b/Open-ILS/web/templates/default/opac/parts/record/extras.tt2
@@ -191,7 +191,7 @@
                 <strong>Patron Reviews:</strong>
                 <div id='rdetail_chilifresh_reviews' class='hide_me' style="margin-bottom:20px;">
                     <div id='chilifreshReviewLink' name='chilifreshReviewLink' class='chili_review'></div>
-                    <div id='chilifreshReviewResult' name='chilifreshReviewResult' style='display:none'></div>
+                    <div id='chilifreshReviewResult' name='chilifreshReviewResult' class='hidden'></div>
                 </div>
                 <strong>Reviews:</strong>
                 <div id='rdetail_review_container' style="margin-bottom:20px;"></div>
diff --git a/Open-ILS/web/templates/default/opac/parts/record/summary.tt2 b/Open-ILS/web/templates/default/opac/parts/record/summary.tt2
index b28cf666fb..5ae513765b 100644
--- a/Open-ILS/web/templates/default/opac/parts/record/summary.tt2
+++ b/Open-ILS/web/templates/default/opac/parts/record/summary.tt2
@@ -27,7 +27,7 @@
                         <td valign="top">
                             <span id='rdetail_title'>[% attrs.title %]</span><br />
                             [% IF attrs.author %]
-                            <span style="color:#545454;">[% l("Author") %]:</span>
+                            <span class='opac-auto-030'>[% l("Author") %]:</span>
                             <em><a title='[% l("Perform an author search") %]'
                                     id='rdetail_author'
                                     href="[% ctx.opac_root %]/results?query=author%3a[% attrs.author | replace('[,\.:;]', '') | uri %]&loc=[% loc %]">[% attrs.author %]</a>
@@ -37,8 +37,7 @@
                         <td align="right" valign="top" nowrap="nowrap" style="white-space:nowrap;">
                             <div style="width:230px;text-align:left;margin-top:3px;">
                                 <div style="float:right;">
-                                    <div style="border-bottom:1px dotted #ccc;padding-top:10px;"
-                                        class="rdetail_aux_utils">
+                                    <div class="rdetail_aux_utils opac-auto-010">
                                         <a href="[% ctx.opac_root %]/place_hold?hold_target=[% record.id %]&hold_type=T" id="rdetail_place_hold"><img
                                             src="[% ctx.media_prefix %]/images/green_check.png" alt="place hold" />
                                             <span style="position:relative;top:-3px;left:3px;">Place Hold</span></a>
@@ -62,7 +61,7 @@
                         </td>
                     </tr>
                 </table>
-                <div style="border-top: 1px dotted #ccc;padding-top:17px;">
+                <div class='opac-auto-018'>
                     <table border="0" cellpadding="0" width="100%">
                         <tr>
                             <td nowrap='nowrap' valign="top">
diff --git a/Open-ILS/web/templates/default/opac/parts/result/table.tt2 b/Open-ILS/web/templates/default/opac/parts/result/table.tt2
index 0e77bbb1cc..58d4f638cd 100644
--- a/Open-ILS/web/templates/default/opac/parts/result/table.tt2
+++ b/Open-ILS/web/templates/default/opac/parts/result/table.tt2
@@ -69,8 +69,7 @@
                 <div style="width:174px;" class="hide_me" id="tehSideBar">SIDEBAR TODO
                 </div>
             </td>
-            <td style="border-left:1px solid #e9ebf3;padding-right:27px;"
-                width="1"></td>
+            <td class='opac-auto-015' width="1"></td>
             <td valign="top">
                 <table id="res_table" cellpadding="0" cellspacing="0"
                     border="0" width="100%" style="margin-top:10px;">
@@ -177,8 +176,7 @@
                                                 class='result_table_format_cell' align="center" width="1"
                                                 style="padding:0px 10px;">
                                                 <img alt="Format" src="" class="hide_me" name="" />
-                                                <span class='hide_me'
-                                                    style='color: #9999FF; padding-left: 10px; font-size: 7pt; font-weight: 300;'>
+                                                <span class='hide_me opac-auto-031'>
                                                     <span>[% l("Match Score: ") %] </span>
                                                     <span name='relevancy_span'> </span>
                                                 </span>
@@ -198,14 +196,13 @@
                                             <td nowrap='nowrap' width="1" align="right">
                                                 <div style="width:250px;text-align:left;">
                                                     <div style="float:right;">
-                                                        <div style="border-bottom:1px dotted #ccc;padding-top:10px;"
-                                                            class="results_aux_utils"><a
+                                                        <div class="results_aux_utils opac-auto-010"><a
                                                                 href="[% ctx.opac_root %]/place_hold?hold_target=[% rec.bre.id %]&hold_type=T" name="place_hold_link"><img
                                                                 src="[% ctx.media_prefix %]/images/green_check.png"
                                                                 alt="place hold" /><span
                                                                     style="position:relative;top:-3px;left:3px;">Place Hold</span></a>
                                                         </div>
-                                                        <div style="border-bottom:1px dotted #ccc;padding-top:6px;" class="results_aux_utils">
+                                                        <div class="results_aux_utils opac-auto-011">
                                                             <div style="position:absolute;">
                                                                 <div style="position:relative;top:5px; left: 25px;">
                                                                     <a title="Add to my list"
@@ -251,7 +248,7 @@
                                         <tr>
                                             <td colspan="5">
                                                 <div style="height:0px;border-top:1px solid #b7b7b7;border-bottom:1px solid #d4d4d4;margin:15px 0px;"></div>
-                                                <!-- <hr style="border-bottom:none;*height:0px;" color="#FFCC33" /> -->
+                                                <!-- <hr class='opac-auto-013' color="#FFCC33" /> -->
                                             </td>
                                         </tr>
                                     </tbody>
-- 
2.11.0