Manage parts holds warning (and removal) w/ CSS

Use add/removeCSSClass to apply and remove holds parts selector
warning instead of inline style.border management, which (when using
'auto') causes problems in IE.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>

diff --git a/Open-ILS/web/opac/skin/default/js/holds.js b/Open-ILS/web/opac/skin/default/js/holds.js
index 72a1946..0363e5b 100644 (file)
--- a/Open-ILS/web/opac/skin/default/js/holds.js
+++ b/Open-ILS/web/opac/skin/default/js/holds.js
@@ -126,7 +126,7 @@ function holdsDrawEditor(args) {
        if(holdArgs.editHold) // flesh the args with the existing hold 
                holdArgsFromHold(holdArgs.editHold, holdArgs);
 
-//    $('holds_parts_selector').style.border = 'auto';
+     removeCSSClass($('holds_parts_selector'), 'parts-warning');
     holdArgs.partsSuggestionMade = false;
 
        holdsDrawWindow();
@@ -1026,7 +1026,7 @@ function holdHandleCreateResponse(r, recurse) {
                         holdArgs.recordParts.length && holdArgs.type == 'T') {
                     // T holds on records that have parts are OK, but if the record has no non-part
                     // copies, the hold will ultimately fail.  Suggest selecting a part to the user.
-                    $('holds_parts_selector').style.border = '2px solid red';
+                    addCSSClass($('holds_parts_selector'), 'parts-warning');
                     holdArgs.partsSuggestionMade = true;
                     alert($('hold_has_parts').innerHTML);
                 } else {

diff --git a/Open-ILS/web/opac/theme/default/css/colors.css b/Open-ILS/web/opac/theme/default/css/colors.css
index db6ba1d..7bd54cc 100644 (file)
--- a/Open-ILS/web/opac/theme/default/css/colors.css
+++ b/Open-ILS/web/opac/theme/default/css/colors.css
@@ -135,3 +135,5 @@ a:focus { background: #E0F0E0; color: #000000;}
 .hold_expire_warning { color:red; }
 
 #result_low_hits { border-top: 2px solid #A7EA9D }
+
+.parts-warning { border: 2px solid red; }