Ouch!

On 2009-07-22, at 16:18EDT, Max Carlson wrote:

> It was a lot of work.  I implemented it for webtop's smartobjects  
> feature. They just decided they don't always want custom context  
> menus...
> -----Original Message-----
> From: P T Withington <[email protected]>
> Sent: Wednesday, July 22, 2009 1:49 PM
> To: Max Carlson <[email protected]>
> Cc: [email protected]; [email protected]; 
> [email protected]
> Subject: Re: [Laszlo-reviews] For Review: Change 20090722-maxcarlson- 
> N Summary: Allow native context menus to be shown for the html tag
>
> So, I'm confused.  Why are we trying to intercept mouse-clicks at all
> in an <html> element?  This seems like a lot of work to intercept them
> and forward them on...
>
> On 2009-07-22, at 15:22EDT, Max Carlson wrote:
>
>> Change 20090722-maxcarlson-N by [email protected] on 2009-07-22
>> 10:21:10 PDT
>>    in /Users/maxcarlson/openlaszlo/trunk-clean
>>    for http://svn.openlaszlo.org/openlaszlo/trunk
>>
>> Summary: Allow native context menus to be shown for the html tag
>>
>> Bugs Fixed: LPP-8298 - In DHTML the html component should show the
>> iframes right click menu, not the lzx right click menu
>>
>> Technical Reviewer: hminsky
>> QA Reviewer: mdemmon
>>
>>
>> Details: lps/components/extensions/html.lzx - Add
>> shownativecontextmenu attribute and setter, clarify documentation for
>> mouseevents setter.
>>
>> iframemanager.js - Add setShowNativeContextMenu() to receive and  
>> store
>> state from the html component.  Always show native context menus if
>> set to true for a given iframe.  Don't forward oncontextmenu events  
>> to
>> Flash runtimes.
>>
>> examples/extensions/html.lzx - Add button that toggles native context
>> menus on/off to test the functionality.
>>
>> Tests: examples/extensions/html.lzx - Clicking 'Toggle native context
>> menus' works in dhtml.  Clicking the 'Toggle native context menus'
>> button in swf warns if the debugger is on.  Right-clicking no longer
>> warns about unhandled events in swf.
>>
>> Files:
>> M      lps/components/extensions/html.lzx
>> M      lps/includes/source/iframemanager.js
>> M      examples/extensions/html.lzx
>>
>> Changeset: 
>> http://svn.openlaszlo.org/openlaszlo/patches/20090722-maxcarlson-N.tar
>> _______________________________________________
>> Laszlo-reviews mailing list
>> [email protected]
>> http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews
>
>

_______________________________________________
Laszlo-reviews mailing list
[email protected]
http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews

Reply via email to