[open-ils-commits] [GIT] Evergreen ILS branch master updated. 40f490e4e91f54297df4b54c0d16c9bc998ad327

[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, master has been updated
       via  40f490e4e91f54297df4b54c0d16c9bc998ad327 (commit)
       via  4231369f29d3eaac461d1de247d0242b8eed7f91 (commit)
      from  131f06e59e4a42a608b2246e60caff8ada673ea1 (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 40f490e4e91f54297df4b54c0d16c9bc998ad327
Merge: 131f06e 4231369
Author: Lebbeous Fogle-Weekley <lebbeous at esilibrary.com>
Date:   Mon Jul 25 11:55:36 2011 -0400

    Merge branch 'parts-hold-warning-fix'


commit 4231369f29d3eaac461d1de247d0242b8eed7f91
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 72a1946..0363e5b 100644
--- 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
--- 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