Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Redux refactoring tracking issue #8528

Closed
47 of 51 tasks
NejcZdovc opened this issue Apr 27, 2017 · 1 comment
Closed
47 of 51 tasks

Redux refactoring tracking issue #8528

NejcZdovc opened this issue Apr 27, 2017 · 1 comment

Comments

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Apr 27, 2017

General

Navigator

@bridiver is working on urlBar related things

Frame

Tabs

Bookmarks

Other components

Disabled unit tests

  • Frame (test/unit/js/components/frameTest.js)

Impure components (contains non primitive types in mergeProps)

  • Explore possibility to remove template prop in MenuBar
  • Explore possibility to remove submenu prop in MenuBarItem
  • Refactor allSiteSettings in Frame component
  • Refactor dnd for tabs (don't send the whole frame in)
@NejcZdovc
Copy link
Contributor Author

closing as this was done

@bbondy bbondy added this to the 0.20.x (Release Channel) milestone Jan 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.