[Laszlo-dev] For Review: Change 20090213-maxcarlson-v Summary: Fix contextmenus for text, clickable views and offset views

[André Bargull]

Walking up the parent-chain doesn't really work (even when the view's 
bounds are checked), does it? For example in this testcase, when you 
right click on the red-view, you should get the "foo"-contextmenu (like 
in swf). But in dhtml, the "bar"-view will receive the mouse-event, so 
looking up the parent-chain will only find the default contextmenu on 
canvas, but not the red view's contextmenu:

> <canvas>
>   <view width="100" height="100" bgcolor="red">
>     <contextmenu>
>       <contextmenuitem caption="foo" />
>     </contextmenu>
>   </view>
>   <view id="bar" width="100" height="100" />
> </canvas>


On 2/14/2009 8:56 AM, Max Carlson wrote:
> Change 20090213-maxcarlson-v by maxcarlson at Bank.lan on 2009-02-13 
> 22:25:50 PST
>     in /Users/maxcarlson/openlaszlo/trunk-clean
>     for http://svn.openlaszlo.org/openlaszlo/trunk
> 
> Summary: Fix contextmenus for text, clickable views and offset views
> 
> Bugs Fixed: LPP-7661 - Views without bgcolor allow context menus to 
> receive clicks in SWF but not in DHTML
> 
> Technical Reviewer: andre.bargull at udo.edu
> QA Reviewer: promanik
> 
> Release Notes:
> 
> Details: LzSprite - Show context menus when clickable sprites get an 
> onmousedown event with event.button == 2 (the context button).
> 
> LzTextSprite - Set owner property of scrolldiv so context menus work.
> 
> LzMouseKernel - Move context menu handling to __showContextMenu().  Walk 
> up the sprite hierarchy looking for sprites with a context menu that 
> have the mouse over them.  If found, show the context menu, otherwise 
> hide any that are showing.
> 
> Tests: See LPP-7661.
> 
> Files:
> M      WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
> M      WEB-INF/lps/lfc/kernel/dhtml/LzTextSprite.js
> M      WEB-INF/lps/lfc/kernel/dhtml/LzMouseKernel.js
> 
> Changeset: 
> http://svn.openlaszlo.org/openlaszlo/patches/20090213-maxcarlson-v.tar
>