Closed Bug 1362059 Opened 3 years ago Closed 2 years ago

Implement RequestListHeaderContextMenu React component

Categories

(DevTools :: Netmonitor, enhancement, P3)

enhancement

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()
Priority: P2 → P3
Summary: Turn request-list-header-context-menu into a react component → Implement RequestListHeaderContextMenu React component
Assignee: nobody → vkatsikaros
Status: NEW → ASSIGNED
Whatever React component we end up with, it would be nice not to lose the native styling the Menu API gives us.
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
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: 2 years ago
Resolution: --- → WONTFIX
Whiteboard: [netmonitor]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.