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 [EMAIL PROTECTED] 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

Reply via email to