Closed Bug 1448810 Opened 4 years ago Closed 4 years ago
Rename the Places sidebar files and merge sidebar
Utils into Places UIUtils
59 bytes, text/x-review-board-request
I'd like to clarify a bit the content files naming bookmarksPanel => bookmarksSidebar history-panel => historySidebar and there's no big point into having sidebarUtils, it can be merged into PUIU and uidensity into the respective sidebar init.
I'd probably also merge organizer.css into places.css, since it only includes one rule (that may even not be necessary or wrong, it should probably be a max-width)
Comment on attachment 8962539 [details] Bug 1448810 - Rename the Places sidebar files and merge sidebarUtils into PlacesUIUtils. https://reviewboard.mozilla.org/r/231344/#review237224 Much nicer and easier to understand.
Attachment #8962539 - Flags: review?(standard8) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/84408e8f7f93 Rename the Places sidebar files and merge sidebarUtils into PlacesUIUtils. r=standard8
You need to log in before you can comment on or make changes to this bug.