[open-ils-commits] [GIT] Evergreen ILS branch master updated. 6fbaf6cec686cfe25b2186799ed7b0368aa1cc6f

[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  6fbaf6cec686cfe25b2186799ed7b0368aa1cc6f (commit)
      from  70d25de2bdd7837f3740769495655b9922e6397c (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 6fbaf6cec686cfe25b2186799ed7b0368aa1cc6f
Author: Steven Chan <schan at sitka.bclibraries.ca>
Date:   Thu May 23 15:22:15 2013 -0700

    Patron Editor can enter erroneous values for Claims-returned count
    
    In the staff client, Patrons interface, Edit screen, when
    mouse-scrolling up and down the Edit form, it is possible to
    inadvertently change the value in Claims-returned Count or Claims Never
    Checked Out Count field.  It will happen if the mouse hovers over either
    data field while using the mouse wheel.
    
    Both data fields use the dijit.form.NumberSpinner widget to provide a
    'spin' behaviour.  The fix involves cancelling the mouse scroll event as
    it propagates to input fields in table rows that are using the
    dijit.form.NumberSpinner widget, because the mouse scroll would be
    applied too early by the widget, before the user has intentionally
    focussed on one of the input fields.  Now, mouse scrolling has no effect
    on the two input fields; it will only have an effect in scrolling the
    page up or down.
    
    Signed-off-by: Steven Chan <schan at sitka.bclibraries.ca>
    Signed-off-by: Ben Shum <bshum at biblio.org>

diff --git a/Open-ILS/web/js/ui/default/actor/user/register.js b/Open-ILS/web/js/ui/default/actor/user/register.js
index b62c3b6..58c2c70 100644
--- a/Open-ILS/web/js/ui/default/actor/user/register.js
+++ b/Open-ILS/web/js/ui/default/actor/user/register.js
@@ -300,6 +300,17 @@ function load() {
     }
         
     uUpdateContactInvalidators();
+
+    // Cancel mouse scroll events from propagating to table rows which are
+    // using the dijit.form.NumberSpinner widget, because the mouse scroll
+    // would be applied too early by the widget, before the user has
+    // intentionally focussed on the input field.
+    dojo.query('tr[wclass="dijit.form.NumberSpinner"]').forEach(function (tr) {
+        tr.addEventListener('DOMMouseScroll', function (ev) {
+            ev.stopPropagation();
+        }, true);
+    });
+
     lock_ready = true;
 }
 

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

Summary of changes:
 Open-ILS/web/js/ui/default/actor/user/register.js |   11 +++++++++++
 1 files changed, 11 insertions(+), 0 deletions(-)


hooks/post-receive
-- 
Evergreen ILS