Looks reasonable to me. Approved. On 2009-11-10, at 06:05, Max Carlson wrote:
> Change 20091110-maxcarlson-4 by maxcarl...@bank on 2009-11-10 02:55:21 > PST > in /Users/maxcarlson/openlaszlo/trunk-clean > for http://svn.openlaszlo.org/openlaszlo/trunk > > Summary: Fix scrollbars to automatically work with the mouse wheel > > Bugs Fixed: LPP-8131 - Mouse Scroll in a grid > > Technical Reviewer: hminsky > QA Reviewer: ptw > > Details: LaszloView - Add isMouseOver() method and dependency. > > newcontent - Scroll for mousewheel events when the mouse is over the > debugger. > > basescrollbar - Update comment for focusview. Move usemousewheel > setter from oninit method. Update listener for mousewheel events to > check if the mouse is over the scrolltarget's parent - and if so, > automatically scroll. > > basegrid - Remove an extra getMouse() call from _setHiliteview(). > > Tests: Use mousewheel over the debugger and the scrollable area in > LPP-8131 > > Files: > M WEB-INF/lps/lfc/views/LaszloView.lzs > M lps/components/debugger/newcontent.lzx > M lps/components/base/basescrollbar.lzx > M lps/components/base/basegrid.lzx > > Changeset: > http://svn.openlaszlo.org/openlaszlo/patches/20091110-maxcarlson-4.tar > _______________________________________________ > Laszlo-reviews mailing list > [email protected] > http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews _______________________________________________ Laszlo-reviews mailing list [email protected] http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews
