[Laszlo-dev] For Review: Change 20090728-maxcarlson-L, Summary: Size views with no bgcolor or resource to 0x0, add separate, tree for context menus
André Bargull
andre.bargull at udo.edu
Thu Jul 30 13:05:10 PDT 2009
This is actually the same issue as LPP-8218, which is caused by the
workaround for LPP-8189.
See this code in kernel/dhtml/LzContextMenuKernel.lzs:
> /** @access private */
> function __show () {
> var owner = this.owner;
> if (this.__sentshowevent == false) {
> // Avoid sending onmenuopen events more than once - see LPP-8189
> this.__sentshowevent = true;
> var del = this._delegate;
> if (del != null) del.execute(owner);
> if (owner.onmenuopen.ready) owner.onmenuopen.sendEvent(owner);
> }
If you re-open the context-menu, "__sentshowevent" is still true and
therefore neither the onmenuopen-event nor the delegate is called..
> Sorry, I don't have a lot of insight to offer. I'd take a look at
> LzMouseKernel.js and track it down from there. It's not a regression -
> it's behavior that's been there for a long time... Henry Minsky wrote:
>> > Hey Max, while you're in the middle of updating the dhtml context menu
>> > code, there's this bug
>> >
>> > http://jira.openlaszlo.org/jira/browse/LPP-8359
>> >
>> >
>> > COntext menu delegate not firing on right click in menu already open
>> >
>> >
>> > I was wondering if you have any hunch as to what the issue would be that
>> > keeps a context menu from
>> > calling it's delegate if a context menu is already open someplace?
>> >
>> >
>> >
>>
>
> -- Regards, Max Carlson OpenLaszlo.org
More information about the Laszlo-dev
mailing list