1029343 - Enable pages to override context menu

Status: RESOLVED INVALID

(Firefox OS Graveyard :: Gaia::System::Browser Chrome, defect)

Description

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

It would be good to enable pages to replace the builtin context menu rather than just add to it. I could have sworn that we had that ability in desktop firefox, but now I can't find it.

I know I argued for that to be added to the spec, but there was little interest back then. But given that apps can completely disable context menus (by calling preventDefault() on the contextmenu event), it's better that apps use <menu> to replace the context menu rather than build their own menu in using <div>s and CSS.

So I propose that we add a "nodefault", or something similar, attribute to <menu> which makes the menu override, rather than add to, the system context menu.

Likely the way to implement this is to make the mozbrowsercontextmenu event include a flag indicating if "nodefault" was set on the <menu> or not.

Comment 1

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Btw, I don't think we should do this only for apps. Given that all pages can disable context menus using the contextmenu event, I don't see a reason to allow it for all webpages.

Comment 2

[Dale Harvey (:daleharvey)]

Closed as with https://bugzilla.mozilla.org/show_bug.cgi?id=1031475