[Laszlo-dev] For Review: Change 20070226-maxcarlson-U Summary: Add support for context menus in DHTM

[J Crowley]

Approved here as well, but you might want to fix the double-right- 
click error.  I'm getting something different from Phil, though:  If  
you right-click (or, in OS X, ctrl-click) on the menu after it's been  
opened, you get the following Firebug errors:

$3.owner has no properties  (LFCdhtml.js (line 1555))

target has no properties  (api.lzx (line 223))

It's not really a show-stopper or anything, and it seems to continue  
to work fine after these errors occur.

On Feb 26, 2007, at 10:22 PM, Max Carlson wrote:

> Change 20070226-maxcarlson-U by maxcarlson at max-carlsons- 
> computer.local on 2007-02-26 19:15:24 PST
>     in /Users/maxcarlson/openlaszlo/legals
>
> Summary: Add support for context menus in DHTM
>
> New Features:
>
> Bugs Fixed: LPP-3573 - Fix LzContextMenu in DHTML
>
> Technical Reviewer: promanik
> QA Reviewer: jcrowley
> Doc Reviewer: (pending)
>
> Documentation:
>
> Release Notes:
>
> Details: api.lzx - remove stale debugger include code
>
> LzContextMenu.lzs - Add implementation of LzContextMenu for DHTML.
>
> LzSprite.js - Implement setDefaultContextMenu() and setContextMenu()
>
> LzMouseKernel.js - Find target for context menu clicks, and show  
> appropriate context menu.
>
> dhtml/Library.lzs - Add LzContextMenu.lzs
>
> LzPlatform.lzs - move to shared location for both swf, dhtml
>
> views/Library.lzs - Include shared LzPlatform.lzs
>
> html-response.xslt - Include style and div for lzcontextmenu in DHTML
>
> Tests: http://localhost:8080/legals/test/contextmenu/api.lzx? 
> lzr=dhtml passes in dhtml, all other apps show default context menu  
> in dhtml.
>
> Files:
> M      test/contextmenu/api.lzx
> A      WEB-INF/lps/lfc/kernel/dhtml/LzContextMenu.lzs
> M      WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
> M      WEB-INF/lps/lfc/kernel/dhtml/LzMouseKernel.js
> M      WEB-INF/lps/lfc/kernel/dhtml/Library.lzs
> D      WEB-INF/lps/lfc/views/platform/swf/LzPlatform.as
> A  +   WEB-INF/lps/lfc/views/platform/LzPlatform.lzs
> D      WEB-INF/lps/lfc/views/platform/dhtml/LzPlatform.js
> M      WEB-INF/lps/lfc/views/Library.lzs
> M      WEB-INF/lps/templates/html-response.xslt
>
> Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20070226- 
> maxcarlson-U.tar