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

[André Bargull]

Ok, I've filed the bug at 
https://bugzilla.mozilla.org/show_bug.cgi?id=481468.
Hopefully we don't get more Firefox bugs today, three should be enough 
for one day! ;-)



On 3/4/2009 7:42 PM, Henry Minsky wrote:
> 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
>>
>>
> 
> 
>