Closed
Bug 1362059
Opened 8 years ago
Closed 7 years ago
Implement RequestListHeaderContextMenu React component
Categories
(DevTools :: Netmonitor, enhancement, P3)
DevTools
Netmonitor
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: rickychien, Unassigned)
References
Details
There are some leftovers modules living in src folder haven't rewrite yet. request-list-header-context-menu.js should turn into a react component and connect to redux store.
Tasks:
* Turn request-list-header-context-menu.js module into a react component
* Hook up the react component to redux store by calling react-redux's connect()
Reporter | ||
Updated•8 years ago
|
Priority: P2 → P3
Summary: Turn request-list-header-context-menu into a react component → Implement RequestListHeaderContextMenu React component
Updated•8 years ago
|
Assignee: nobody → vkatsikaros
Status: NEW → ASSIGNED
Comment 1•8 years ago
|
||
Whatever React component we end up with, it would be nice not to lose the native styling the Menu API gives us.
Comment 2•7 years ago
|
||
This required for me lots of react/redux reading and testing, and so far it took me more than 3 weeks to produce any solid results. So I am leaving this in case someone wants to pick it up and finish it. I'll just continue it in the background (if I finish it before someone else gets there it's fine).
Assignee: vkatsikaros → nobody
Status: ASSIGNED → NEW
Reporter | ||
Comment 3•7 years ago
|
||
Close this issue due to agreement in today's netmonitor meeting due to https://bugzilla.mozilla.org/show_bug.cgi?id=1336382#c2.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Whiteboard: [netmonitor]
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•