[Laszlo-dev] For Review: Change 20090304-bargull-JfK Summary: workaround for Firefox DOM-focus bug

[Henry Minsky]

approved!

I don't have a mozilla bug account, so please file your test case with them.

And thanks for finding and fixing this, you're like the cavalry riding
in to save the day!


On Wed, Mar 4, 2009 at 1:24 PM, André Bargull <andre.bargull at udo.edu> wrote:
> Change 20090304-bargull-JfK by bargull at dell--p4--2-53 on 2009-03-04 19:03:59
> in /home/Admin/src/svn/openlaszlo/trunk
> for http://svn.openlaszlo.org/openlaszlo/trunk
>
> Summary: workaround for Firefox DOM-focus bug
>
> New Features:
>
> Bugs Fixed: LPP-7786 (It takes two clicks to get focus into an
> edittext/inputtext in DHTML)
>
> Technical Reviewer: hminsky
> QA Reviewer: mdemmon
> Doc Reviewer: (pending)
>
> Documentation:
>
> Release Notes:
>
> Details:
> DOM operations on a blurring element breaks the focus in Firefox. It's
> possible to reproduce the same bug with plain HTML and Javascript, I've
> attached a testcase at the bugreport. (Henry, do you've got an account for
> their Bugzilla to report the issue? Otherwise I need to create an account.)
> To workaround that bug, I've added a new quirk "dom_breaks_focus" which is
> enabled for Firefox. If the quirk is enabled and DOM operations were
> performed (__shown property changes from 'true' to 'false'), focus() needs
> to be called manually on the "__focusedSprite". This is done at the end of
> "__textEvent()" to ensure that "onfocus" won't be send too early.
>
>
> Tests:
> testcase from bugreport
>
> Files:
> M WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
> M WEB-INF/lps/lfc/kernel/dhtml/LzInputTextSprite.js
>
> Changeset:
> http://svn.openlaszlo.org/openlaszlo/patches/20090304-bargull-JfK.tar
>
>



-- 
Henry Minsky
Software Architect
hminsky at laszlosystems.com