search

firebug / firebug.next

Next Firebug generation built on top of native Firefox developer tools
Other
229 stars 37 forks source link

Use fluxify for XHR viewer #411

Closed janodvarko closed 9 years ago

janodvarko commented 9 years ago

User proper architecture for XHR viewer (in the Console panel)

https://github.com/mozilla/gecko-dev/tree/fx-team/browser/devtools/shared/fluxify https://github.com/mozilla/gecko-dev/tree/fx-team/browser/devtools/debugger/content

Honza

jsantell commented 9 years ago

We are replacing fluxify with redux, so we have good documentation, don't have to maintain our own "fork" of it, and everything that fluxify was doing is possible with vanilla redux -- this should land today or so: https://bugzilla.mozilla.org/show_bug.cgi?id=1204173

let me know if you have any questions and I can help with porting, shouldn't be that bad, mostly just method name changes and using a reducer to replicate the emitter being passed into other reducers

janodvarko commented 9 years ago

This is great news, I actually wanted to rather use Redux! As soon as I make more progress on this, I'd love to have a discussion about whether I am going in the right direction, to make sure we are on the same page.

I want to port this feature to devtools eventually.

Honza

janodvarko commented 9 years ago

New issue reported: #412

Honza