[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