Open Bug 1297412 Opened 3 years ago Updated Last year

Implement HTML Popup Menu

Categories

(DevTools :: Framework, defect, P3)

defect

Tracking

(Not tracked)

People

(Reporter: Honza, Unassigned)

References

(Depends on 1 open bug)

Details

(Whiteboard: [reserve-html])

Some parts of the DevTools UI requires a popup menu.

E.g.
* IFrame picker command button on Toolbox toolbar
* All-tabs-Menu for Inspector panel sidebar (bug 1281789)

The current Menu component (client/framework/menu) is based on <xul:menupopup> and so, represents a blocker in our XUL -> HTML effort.

We should implement a new HTML based component supporting the same interface as the current Menu.

The component can utilize existing tooltip api, HTML5 context menu or React. We should start with a simple solution.

Honza
Whiteboard: [devtools-html]
Note that we can and should continue using the Menu API even in HTML docs running in chrome scope. We will just get an error when it's opened if running in content priv, which is what this bug will address
Depends on: 1257613
(In reply to Jan Honza Odvarko [:Honza] from comment #0)
> Some parts of the DevTools UI requires a popup menu.
> 
> The component can utilize existing tooltip api, HTML5 context menu or React.
> We should start with a simple solution.
> 

Just for reference the html Tooltip API implemented for devtools-html is at HTMLTooltip.js [1].

[1] http://searchfox.org/mozilla-central/source/devtools/client/shared/widgets/HTMLTooltip.js
Flags: qe-verify+
Priority: -- → P2
QA Contact: cristian.comorasu
Whiteboard: [devtools-html] → [reserve-html]
Helen(UX) call this as `additional menu` in bug 1292054

It also appears in

* All-tabs-Menu for Network monitor panel sidebar

Since the sidebar tabs are formed by react, we might refer http://instructure-react.github.io/react-menu/ to make our version of Popup/additional Menu
Depends on: 1292054
Blocks: devtools-html-phase2
No longer blocks: devtools-html-2
Whiteboard: [reserve-html] → [devtools-html]
Priority: P2 → P1
Priority: P1 → P2
Blocks: 1302129
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Iteration: --- → 52.1 - Oct 3
Priority: P2 → P1
Assignee: jdescottes → nobody
Status: ASSIGNED → NEW
Iteration: 52.1 - Oct 3 → ---
Priority: P1 → P2
Priority: P2 → P1
Priority: P1 → P3
Whiteboard: [devtools-html] → [reserve-html]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.