1448810 - Rename the Places sidebar files and merge sidebarUtils into PlacesUIUtils

Status: RESOLVED FIXED

(Firefox :: Bookmarks & History, enhancement, P3)

Description

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

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.

Comment 1

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

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 2

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

Attachment: Bug 1448810 - Rename the Places sidebar files and merge sidebarUtils into PlacesUIUtils.

Review commit: https://reviewboard.mozilla.org/r/231344/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/231344/

Comment 3

[Mark Banner (:standard8)]

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.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/84408e8f7f93
Rename the Places sidebar files and merge sidebarUtils into PlacesUIUtils. r=standard8

Comment 5

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/84408e8f7f93