TPAC: Address remaining colors and background URL image

Thanks to Thomas Berezansky for finding these remaining hard-coded CSS
colors, and raising a question about one of the background images that
turns out to not currently being in use.

Signed-off-by: Dan Scott <dan@coffeecode.net>
Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>

diff --git a/Open-ILS/src/templates/opac/css/style.css.tt2 b/Open-ILS/src/templates/opac/css/style.css.tt2
index 5ed8ccb..07300f8 100644 (file)
--- a/Open-ILS/src/templates/opac/css/style.css.tt2
+++ b/Open-ILS/src/templates/opac/css/style.css.tt2
@@ -356,7 +356,7 @@ h2.rdetail_uris {
 }
 div.rdetail_uris {
     padding: 0.5em 1em 0.5em 1em;
-    background-color: lightGray;
+    background-color: [% css_colors.accent_lighter2 %];
 }
 div.rdetail_uris ul li {
     position: relative;
@@ -486,18 +486,7 @@ div.format_icon {
     color: [% css_colors.text_invert %];
 }
 
-#paginate-homebanner a.toc {
-    display:block;
-    width:20px;
-    height:20px;
-    background:gray;
-    float:left;
-    margin-left:2px;
-    margin-right:2px;
-    margin-top:2px;
-}
-
-#rdetail_extras_expand, #rdetail_extras_collapse, #rdetail_locs_collapse {
+##rdetail_extras_expand, #rdetail_extras_collapse, #rdetail_locs_collapse {
     margin-left: 13px;
 }
 
@@ -511,13 +500,6 @@ div.format_icon {
     margin:0;
 }
 
-#paginate-homebanner a.selected {
-    border: 2px solid [% css_colors.border_dark %];
-    margin-top:0px;
-    margin-left:0px;
-    margin-right:0px;
-}
-
 .almost-content-wrapper {
     background: [% css_colors.background %];
 }
@@ -593,11 +575,6 @@ div.format_icon {
     padding-top: 1em;
 }
 
-#home-buttons-inner {
-    height:117px;
-    background:#bda964;
-}
-
 #holds_temp_parent td {
     border-bottom:1px solid [% css_colors.border_standard %];
 }
@@ -892,7 +869,8 @@ table.acct_notes th {
 }
 
 .payment-processing {
-    font-weight: bold; color: green;
+    font-weight: bold;
+    color: [% css_colors.text_greatnews %];
     font-size: 120%;
     padding: 10px; border: 1px solid [% css_colors.accent_medium_dark %];
     text-align: center;
@@ -928,7 +906,13 @@ table.acct_notes th {
 .facetFieldLabel { padding-left: 2px; margin-top: 5px; margin-bottom: 5px; font-weight: bold; text-align: left; }
 .extraFacetFieldsWrapper { }
 .toggleExtraFacetFieldsButton { float: right; margin: 0px; padding: 0px; }
-.facetFieldLineCount { display: inline-block; border-right: 1px solid [% css_colors.accent_light %]; color: gray; width: 3em; margin-right: 3px }
+.facetFieldLineCount {
+    display: inline-block;
+    border-right: 1px solid [% css_colors.accent_light %];
+    color: [% css_colors.accent_mediumdark %];
+    width: 3em;
+    margin-right: 3px
+}
 .facetField { border-top: 1px solid [% css_colors.accent_light %]; }
 .facetFields { padding-left: 5px; }
 .facetFieldLineValue { overflow: hidden; text-overflow: ellipsis; }
@@ -988,10 +972,6 @@ div.facet_sidebar {
     padding-top:6px;
 }
 
-.facet_box_wrapper .box_wrapper .bottom {
-    background:url('/images/facet_box_bg_bottom.png') no-repeat;
-}
-
 .facet_template {
     padding-bottom:5px;
 }
@@ -1003,7 +983,7 @@ div.facet_sidebar {
 
 .facet_template .count {
     float:right;
-    color:#818080;
+    color: [% css_colors.accent_mediumdark %];
     padding-right:11px;
 }
 
@@ -1204,7 +1184,10 @@ a.opac-button {
 .bookbag-item-row td { vertical-align: top; }
 
 .error { color: [% css_colors.text_alert %]; font-weight: bold; }
-.success { color: green; font-weight: bold; }
+.success {
+    color: [% css_colors.text_greatnews %];
+    font-weight: bold;
+}
 
 .rdetail_related_subjects {
     margin-top: 1.5em;
@@ -1401,37 +1384,37 @@ a.preflib_change {
 }
 .popmenu li:hover li a {
     background-color: [% css_colors.primary %]; 
-    color: [% css_colors.accent_ultratint %];
+    color: [% css_colors.accent_ultralight %];
 }
 .popmenu li li a:hover {
-    background-color: [% css_colors.accent_ultratint %]; 
+    background-color: [% css_colors.accent_ultralight %]; 
     color: [% css_colors.primary %];
 }
 /* Styles for the temporary list entry. */
 .popmenu li:hover li[class~="temporary"] a {
     background-color: [% css_colors.primary %]; 
-    color: [% css_colors.accent_ultratint %];
+    color: [% css_colors.accent_ultralight %];
 }
 .popmenu li li[class~="temporary"] a:hover {
-    background-color: [% css_colors.accent_ultratint %]; 
+    background-color: [% css_colors.accent_ultralight %]; 
     color: [% css_colors.primary %];
 }
 /* Styles for the default list entry. */
 .popmenu li:hover li[class~="default"] a {
     background-color: [% css_colors.primary %]; 
-    color: [% css_colors.accent_ultratint %];
+    color: [% css_colors.accent_ultralight %];
 }
 .popmenu li li[class~="default"] a:hover {
-    background-color: [% css_colors.accent_ultratint %]; 
+    background-color: [% css_colors.accent_ultralight %]; 
     color: [% css_colors.primary %];
 }
 /* Styles for the new list entry. */
 .popmenu li:hover li[class~="new"] a {
     background-color: [% css_colors.primary %]; 
-    color: [% css_colors.accent_ultratint %];
+    color: [% css_colors.accent_ultralight %];
 }
 .popmenu li li[class~="new"] a:hover {
-    background-color: [% css_colors.accent_ultratint %]; 
+    background-color: [% css_colors.accent_ultralight %]; 
     color: [% css_colors.primary %];
 }
 /* Style to add a divider on the menu. */

diff --git a/Open-ILS/src/templates/opac/parts/css/colors.tt2 b/Open-ILS/src/templates/opac/parts/css/colors.tt2
index f8a1749..eda641d 100644 (file)
--- a/Open-ILS/src/templates/opac/parts/css/colors.tt2
+++ b/Open-ILS/src/templates/opac/parts/css/colors.tt2
@@ -9,6 +9,7 @@
         text_attention = "#ffcc33", # pumpkin orange
         text_badnews = "#f41d36", # cherry red
         text_goodnews = "#1dd93c", # lime green
+        text_greatnews = "green", # green
         text_invert = "#fff", # white
         text_match = "#c00", # deep red
         primary = "#00593d", # dark green
@@ -20,7 +21,6 @@
         accent_lighter2 = "#d8d8d8", #  grey (lighter again)
         accent_lightest = "#eee", #  grey (lightest)
         accent_ultralight = "#f0f0f0", # ivory
-        accent_ultratint = "#f0e0e0", # ivory-pink
         accent_medium = "#999", # gun metal grey (medium)
         accent_mediumdark = "#888", # gun metal grey (medium dark)
         accent_dark = "#666", # gun metal grey (dark)

diff --git a/Open-ILS/web/images/facet_box_bg_bottom.png b/Open-ILS/web/images/facet_box_bg_bottom.png
deleted file mode 100644 (file)
index fc776c3..0000000
Binary files a/Open-ILS/web/images/facet_box_bg_bottom.png and /dev/null differ

diff --git a/docs/RELEASE_NOTES_NEXT/tpac-css-colors.txt b/docs/RELEASE_NOTES_NEXT/tpac-css-colors.txt
new file mode 100644 (file)
index 0000000..b59ccae
--- /dev/null
+++ b/docs/RELEASE_NOTES_NEXT/tpac-css-colors.txt
@@ -0,0 +1,29 @@
+TPAC: Simplified CSS Color Customization
+========================================
+CSS colors are now defined as a pair of Template::Toolkit files,
+`Open-ILS/src/templates/opac/css/styles.css.tt2` and
+`Open-ILS/src/templates/opac/parts/css/colors.tt2`. Evergreen administrators
+can customize the color scheme for a given skin by copying `colors.tt2` into a
+template override directory and adjusting the colors as desired.
+
+Change required to eg_vhost.conf
+--------------------------------
+To enable Apache to pass the CSS file to the Template::Toolkit handler, you
+must remove `.css` from the list of file extensions that should not be passed
+to a handler in `eg_vhost.conf` as follows:
+
+.From
+------------------------------------------------------------------------------
+<LocationMatch ^/eg/.*(\.js|\.css|\.html|\.xhtml|\.xml|\.jpg|\.png|\.gif)$>
+    SetHandler None
+</LocationMatch>
+------------------------------------------------------------------------------
+
+.To
+------------------------------------------------------------------------------
+<LocationMatch ^/eg/.*(\.js|\.html|\.xhtml|\.xml|\.jpg|\.png|\.gif)$>
+    SetHandler None
+</LocationMatch>
+------------------------------------------------------------------------------
+
+After making this change, restart Apache to make the change take effect.