Change 20080609-Philip-6 by Philip@Philip-DC on 2008-06-09 19:30:39 EDT      in /cygdrive/f/laszlo/svn/src/svn/openlaszlo/trunk      for http://svn.openlaszlo.org/openlaszlo/trunk
Change 20080609-Philip-6 by Philip@Philip-DC on 2008-06-09 19:30:39 EDT
     in /cygdrive/f/laszlo/svn/src/svn/openlaszlo/trunk
     for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: IE7/dhtml: Weather demo does not load

New Features:

Bugs Fixed: LPP-6142

Technical Reviewer: max
QA Reviewer: (pending)
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details:
This line in LzInputTextSprite.select()
     this.__LzInputDiv.focus();

can fail in IE7/dhtml because of an error,

     Can't move focus to the control because it is invisible, not enabled, or of a type that does not accept the focus.

The fix I used was to wrap this line with a try/catch block. I tried to conditionalize the call but I still received the error. Feel free to suggest something better. I think this bug was exposed by LPP-6069.


Tests:
http://localhost:8080/trunk/demos/weather/weather.lzx?lzr=dhtml runs in IE7. It continues to load/run in FF2/dhtml and swf.

Files:
M      WEB-INF/lps/lfc/kernel/dhtml/LzInputTextSprite.js

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20080609-Philip-6.tar



git-svn-id: d62bde4b5aa582fdf07c8d403e53e0399a7ed285@9605 fa20e4f9-1d0a-0410-b5f3-dc9c16b8b17c

mcarlson/openlaszlo,mcarlson/openlaszlo,mcarlson/openlaszlo,mcarlson/openlaszlo,mcarlson/openlaszlo,mcarlson/openlaszlo,mcarlson/openlaszlo