[open-ils-commits] [GIT] Evergreen ILS branch rel_2_1 updated. a8659bf1eae9ccf70b0c050cb070635ec313f884

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_2_1 has been updated
       via  a8659bf1eae9ccf70b0c050cb070635ec313f884 (commit)
      from  efc249c1699f517beddd591f2cfc49a2fb27e017 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit a8659bf1eae9ccf70b0c050cb070635ec313f884
Author: Bill Erickson <berick at esilibrary.com>
Date:   Mon Jul 25 11:38:17 2011 -0400

    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 at esilibrary.com>
    Signed-off-by: Lebbeous Fogle-Weekley <lebbeous at 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 b6e17a2..3bb583b 100644
--- a/Open-ILS/web/opac/skin/default/js/holds.js
+++ b/Open-ILS/web/opac/skin/default/js/holds.js
@@ -109,7 +109,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();
@@ -1009,7 +1009,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
--- 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; }

-----------------------------------------------------------------------

Summary of changes:
 Open-ILS/web/opac/skin/default/js/holds.js     |    4 ++--
 Open-ILS/web/opac/theme/default/css/colors.css |    2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
Evergreen ILS