[Laszlo-dev] For Review: Change 20090611-ptw-8 Summary:, Ensure DHTML app has focus when loaded
P T Withington
ptw at pobox.com
Fri Jun 12 05:30:55 PDT 2009
Thanks for the review. I'll revise my change and resubmit.
On 2009-06-12, at 08:11EDT, André Bargull wrote:
> Why do you focus the window in "lz.embed.__dhtmlLoadScript(..)"?
> That helper function is used to load arbitrary script content
> (excanvas.js, lfc js-file and the application js-file). IMHO
> "lz.embed.dhtml(..)" or the LzSprite ctor would be a better place.
> And in LzModeManager, you need to make that change at two places:
> handleMouseEvent() and __LZallowInput().
>> Change 20090611-ptw-8 by ptw at dueling-banjos.home on 2009-06-11
>> 16:11:19 EDT
>> in /Users/ptw/OpenLaszlo/trunk
>> for http://svn.openlaszlo.org/openlaszlo/trunk
>> Summary: Ensure DHTML app has focus when loaded
>> Bugs Fixed: LPP-8242 OL4.4 - DHTML: The UP, DOWN keys and space
>> bar are invalid for Textlistitem on WinXP/FF3
>> Technical Reviewer: max (pending)
>> QA Reviewer: dju at laszlosystems.com (pending)
>> LzModeManager: Removed some platform-specific code that is no
>> longer necessary now that all platforms use the same debugger.
>> embednew: Restore window focus for Firefox on Windows once the
>> DHTML application is added to the DOM
>> Test case from bug
>> M WEB-INF/lps/lfc/services/LzModeManager.lzs
>> M lps/includes/source/embednew.js
>> Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20090611-ptw-8.tar
More information about the Laszlo-dev