[Laszlo-dev] For Review: Change 20070628-Philip-6. Summary: Fix getContextMenu()

[Max Carlson]

Approved.  I see only one entry added per button click in swf, but 
multiple entries under DHTML.  Can you file a bug?

Philip Romanik wrote:
> Change 20070628-Philip-6 by Philip at Philip-DC on 2007-06-28 11:24:58 EST
>     in /cygdrive/f/laszlo/svn/src/svn/openlaszlo/branches/legals
>     for http://svn.openlaszlo.org/openlaszlo/branches/legals
> 
> Summary: Fix getContextMenu()
> 
> New Features:
> 
> Bugs Fixed: LPP-4155
> 
> Technical Reviewer: max
> QA Reviewer: (pending)
> Doc Reviewer: (pending)
> 
> Documentation:
> 
> Release Notes:
> 
> Details:
> LaszloCanvas.lzs
>   - Added a call to setContextMenu() in setDefaultContextMenu() to match 
> the behavior of 3.4.
> LaszloView.lzs
>   - Removed this.contextMenu (not needed)
>   - getContextMenu() calls the sprite to get the value
> LzSprite.as
>   - Store current context menu in this.__contextmenu.
>   - Added getContextMenu()
> LzSprite.js
>   - Added getContextMenu()
> 
> I found a small dhtml/swf issue that I submitted as LPP-4213.
> 
> Tests:
> http://localhost:8080/legals/test/contextmenu/api.lzx?lzr=dhtml
> smokecheck passes
> 
> This app runs in dhtml and swf:
> 
> <canvas debug="true">
>   <button text="getContextMenu">
>     <handler name="onclick">
>       var cm = canvas.getContextMenu();
>       var del = new LzDelegate (this, 'myHandler');
>       cm.addItem(cm.makeMenuItem('My Function', del));
>       Debug.write(cm);
>     </handler>
> 
>     <method name="myHandler">
>       Debug.write("myHandler");
>     </method>
>   </button>
> </canvas>
> 
> 
> Files:
> M      WEB-INF/lps/lfc/kernel/swf/LzSprite.as
> M      WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
> M      WEB-INF/lps/lfc/views/LaszloView.lzs
> M      WEB-INF/lps/lfc/views/LaszloCanvas.lzs
> 
> Changeset: 
> http://svn.openlaszlo.org/openlaszlo/patches/20070628-Philip-6.tar
> 
> 
> 

-- 
Regards,
Max Carlson
OpenLaszlo.org