You say that "swf_swallows_focus_events"-quirk is necessary for ActiveX 
players, but in "__updateQuirks()" you're just testing for Windows 
platforms:
> +    if (System.capabilities.os.indexOf('Win') == 0) {
> +        quirks.swf_swallows_focus_events = true;
> +    }

Doesn't it work to test playerType for ActiveX?
> if (System.capabilities.playerType == |"ActiveX") {...}|


On 7/8/2009 4:25 AM, P T Withington wrote:
> [I plan to check this in and start a build for testing]
>
> Change 20090707-ptw-J by [email protected] on 2009-07-07  
> 20:37:28 EDT
>      in /Users/ptw/OpenLaszlo/4.2
>      for http://svn.openlaszlo.org/openlaszlo/branches/4.2
>
> Summary: Part 1 of fixing stuck keys when IE loses focus
>
> Bugs Fixed: LPP-8283 Alt tab issue when switching tasks in IE7/Win
>
> Technical Reviewer: max (pending)
> QA Reviewer: hminsky (pending)
>
> Details:
>
>      swf/LzSprite: Create a quirk for ActiveX which for some reason  
> seems
>      to swallow browser focus events, preventing us from using the
>      solution that works for all other platforms of releasing all keys
>      when the the browser window regains focus.
>
>      swf/LzKeyboardKernel: Create a Selection listener for focus change,
>      call __allKeysUp when focus moves to or from the app
>
>      swf9/LFCApplication: For ActiveX embedding, create a Stage
>      listener for ACTIVATE, call __allKeysUp when the app is activated.
>
>      LzKeys, embednew: Add some debug info
>
> Tests:
>      Still doesn't pass the test case, but it will remove the stuck key
>      if you click on an input element
>
> Files:
> M      WEB-INF/lps/lfc/kernel/swf/LzKeyboardKernel.as
> M      WEB-INF/lps/lfc/kernel/swf/LzSprite.as
> M      WEB-INF/lps/lfc/kernel/swf9/LFCApplication.as
> M      WEB-INF/lps/lfc/services/LzKeys.lzs
> M      lps/includes/source/embednew.js
>
> Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20090707-ptw-J.tar
> _______________________________________________
> Laszlo-reviews mailing list
> [email protected]
> http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews
>
>   
_______________________________________________
Laszlo-reviews mailing list
[email protected]
http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews

Reply via email to