Make scrollOnFocus default to false for dojo-based/conify interfaces user/bshum/hammer-for-scrollonfocus
authorLebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Fri, 28 Sep 2012 19:26:06 +0000 (15:26 -0400)
committerBen Shum <bshum@biblio.org>
Tue, 2 Oct 2012 01:55:48 +0000 (21:55 -0400)
commit5386d6108a6aedfde6dac5599a08c69dc0fea74b
treeadbf72d594c2fdccfd618e06d868812149e5c41e
parent2506f44116762d102e02bc190b55d7e82d56c6da
Make scrollOnFocus default to false for dojo-based/conify interfaces

This change is intended to prevent a phenomenon in certain user
interfaces wherein clicking on a button or other widget can cause a
user's screen to suddenly jump and the user to need to click the widget
a second time before the desired function actually happens.

Dijits that have the scrollOnFocus property are the ones that tend to
exhibit this issue, especially, but not always, within the staff client
when the user's xulrunner window is not maximized.

There may be cases of interfaces actually relying on scrollOnFocus to do
something, but I think those cases should be so rare that this is still
worth doing, and exceptions can be applied there as needed.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
Open-ILS/web/js/ui/base.js