When I run the test case with your change applied, I see two gray views.  

In Safari:  Right-click still does nothing on the top view.  On the bottom 
view, I do get the built-in context menu.
In Firefox:  Right-click does nothing on either view
In IE:  Right-click on the top view shows the LZX menu, on the bottom view, I 
get nothing.

This doesn't seem to be an improvement?

On 2010-10-21, at 10:20, Max Carlson wrote:

> Change maxcarlson-20101021-FXR by [email protected] on 2010-10-21 
> 08:15:07 MDT
>    in /Users/maxcarlson/openlaszlo/trunk2
>    for http://svn.openlaszlo.org/openlaszlo/trunk
> 
> Summary: Fix DHTML context menus
> 
> Bugs Fixed: LPP-9442 - regression in DHTML context menu
> 
> Technical Reviewer: ptw
> QA Reviewer: hminsky
> 
> Details: test/contextmenu/api.lzx - Update to eliminate warnings, add test 
> for showBuiltInItems().
> 
> LzMouseKernel - Now that only global oncontextmenu is used, simplify global 
> mouse handler to only have to deal with that kind of event.  Pass contextmenu 
> object directly to __showContextMenu().  
> 
> Tests: test/contextmenu/api.lzx?lzr=dhtml to runs correctly.
> 
> Files:
> M       test/contextmenu/api.lzx
> M       WEB-INF/lps/lfc/kernel/dhtml/LzMouseKernel.js
> 
> Changeset: 
> http://svn.openlaszlo.org/openlaszlo/patches/maxcarlson-20101021-FXR.tar


Reply via email to