[Laszlo-dev] For Review: Change 20090611-ptw-8 Summary:, Ensure DHTML app has focus when loaded
André Bargull
andre.bargull at udo.edu
Fri Jun 12 05:11:55 PDT 2009
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)
>
> Details:
> 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
>
> Tests:
> Test case from bug
>
> Files:
> 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
mailing list