Open
Bug 1297412
Opened 9 years ago
Updated 2 years ago
Implement HTML Popup Menu
Categories
(DevTools :: Framework, defect, P3)
DevTools
Framework
Tracking
(Not tracked)
NEW
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
Reporter | ||
Updated•9 years ago
|
Whiteboard: [devtools-html]
Comment 1•9 years ago
|
||
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
Comment 2•9 years ago
|
||
(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
Updated•9 years ago
|
Blocks: devtools-html-2
Flags: qe-verify+
Priority: -- → P2
QA Contact: cristian.comorasu
Whiteboard: [devtools-html] → [reserve-html]
Comment 3•9 years ago
|
||
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
Updated•9 years ago
|
Updated•9 years ago
|
Whiteboard: [reserve-html] → [devtools-html]
Updated•9 years ago
|
Priority: P2 → P1
Updated•8 years ago
|
Priority: P1 → P2
Updated•8 years ago
|
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Iteration: --- → 52.1 - Oct 3
Priority: P2 → P1
Updated•8 years ago
|
Assignee: jdescottes → nobody
Status: ASSIGNED → NEW
Iteration: 52.1 - Oct 3 → ---
Updated•8 years ago
|
Priority: P1 → P2
Updated•8 years ago
|
Priority: P2 → P1
Updated•8 years ago
|
Priority: P1 → P3
Whiteboard: [devtools-html] → [reserve-html]
Updated•7 years ago
|
Product: Firefox → DevTools
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•