Re: [Laszlo-dev] For Review: Change 20090611-ptw-8 Summary:, Ensure DHTML app has focus when loaded

Fri, 12 Jun 2009 05:08:33 -0700

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 [email protected] 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: [email protected] (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

  






















					Reply via email to