Closed Bug 1178254 Opened 5 years ago Closed 3 years ago

Breakdown: Rewrite toolbox.xul in HTML

Categories

(DevTools :: Framework, enhancement, P1)

enhancement

Tracking

(Not tracked)

RESOLVED FIXED
Iteration:
51.2 - Aug 29

People

(Reporter: pbro, Assigned: bgrins)

References

(Depends on 2 open bugs)

Details

(Whiteboard: [devtools-html])

toolbox.xul is this very thin UI wrapper around tools iframes. In an effort to reduce how many XUL features we use, we should rewrite this document in HTML instead.

Most of the UI is:
- a list of tabs,
- a list of buttons

But there's also:
- the split console
- a context menu
- key shortcuts
- the notification box

Re-doing the tabs and buttons in HTML is fairly easy, however the other part seems more complex to replace.

I believe there are enough advantages in getting rid of XUL to consider doing this anyway.
wip HTML mockup for the toolbar part only: https://dl.dropboxusercontent.com/u/714210/toolbar.html
(supports drag/drop of tabs, but that's not for this bug)
(In reply to Patrick Brosset [:pbrosset] [:patrick] [:pbro] from comment #0)
> But there's also:
> - the split console
> - a context menu
> - key shortcuts
> - the notification box
> 
> Re-doing the tabs and buttons in HTML is fairly easy, however the other part
> seems more complex to replace.
In the short term, for the key shortcuts, the context menu and the split console splitter, we can use xul as additional namespace and xhtml as main namespace. Then we should be able to use xul elements like this : <xul:menu></xul:menu>

As for the notification box, it should be fairly easy to create our custom one in html. That will allow us to have prettier notifications too.
Flags: qe-verify-
Summary: Rewrite toolbox.xul in HTML → Breakdown: Rewrite toolbox.xul in HTML
Whiteboard: [devtools-html]
Severity: normal → enhancement
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
Iteration: --- → 51.1 - Aug 15
Priority: -- → P1
Depends on: 1245921, 1260552, 1266134
Depends on: 1251394
No longer blocks: 1239859
Iteration: 51.1 - Aug 15 → 51.2 - Aug 29
Depends on: 1297109
Depends on: 1297113
Depends on: 1297116
Depends on: 1297132
Done with the initial breakdown
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.