Not approved because "lz.embed.browser" may be "null" (only lz.embed cannot be null, see LzBrowser.lzs), consider running the application with "lzt=swf". And I think the OS' identity is "Mac" not "mac", cf. "dataOS" object in embednew.js.
Approved if you add "lz.embed.browser != null" and the correct case is checked. On 2/23/2010 4:39 AM, Max Carlson wrote: > Change 20100222-maxcarlson-o by maxcarl...@bank on 2010-02-22 19:35:00 PST > in /Users/maxcarlson/openlaszlo/trunk-clean > for http://svn.openlaszlo.org/openlaszlo/trunk > > Summary: Conditionalize textfield focus workaround from LPP-7957 > > Bugs Fixed: LPP-8783 - Text field loses focus when mousing out of canvas and > then mousing back in (SWF9) > > Technical Reviewer: ptw > QA Reviewer: [email protected] > > Details: Conditionalize workaround to only happen for FF< 3.5 on OS X. > > Tests: See LPP-8783. > > Files: > M WEB-INF/lps/lfc/kernel/swf9/LzMouseKernel.as > > Changeset: > http://svn.openlaszlo.org/openlaszlo/patches/20100222-maxcarlson-o.tar > _______________________________________________ Laszlo-reviews mailing list [email protected] http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews
