Remove the Bookmarks sidebar control as a toolbar customization item

RESOLVED FIXED in Firefox 28

Status

()

RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: jaws, Assigned: mconley)

Tracking

(Blocks: 1 bug)

Trunk
Firefox 28
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [Australis:M6])

Attachments

(1 attachment, 1 obsolete attachment)

We now have quite a few bookmark-related items in our customization palette. Bug 855805 is implementing the new bookmarks widget, and thus we shouldn't have multiple bookmark widgets within our customization palette. Having multiple will just lead to user confusion and make guiding users through our support harder.

The Bookmarks item that will be removed is the one that toggles the visibility of the Bookmarks sidebar. We will keep the Bookmarks item that allows the user to view their bookmarks in a panel/popup as well as convenient access to bookmarking the current page.

This bug is not concerned with removing the bookmarks sidebar.
Component: Migration → Toolbars
is there an equivalent for the history button that opens the sidebar?
i suppose the same reasoning applies

Updated

6 years ago
Summary: Remove the "Bookmarks" control as a possible customization item → Remove the Bookmarks sidebar control as a toolbar customization item
(In reply to Marco Bonardo [:mak] from comment #1)
> is there an equivalent for the history button that opens the sidebar?
> i suppose the same reasoning applies

Thanks, I just filed that as bug 868135.
(Assignee)

Updated

6 years ago
No longer blocks: 860814
Whiteboard: [Australis:M6]
(Assignee)

Updated

6 years ago
Blocks: 872617
(Assignee)

Updated

6 years ago
Assignee: nobody → mconley
(Assignee)

Comment 3

6 years ago
Created attachment 753900 [details] [diff] [review]
Patch v1

Here's a first pass at this. Am I forgetting anything?
Attachment #753900 - Flags: review?(jaws)
Comment on attachment 753900 [details] [diff] [review]
Patch v1

You forgot to remove the bookmarksButton.tooltip string and bookmarksButtonObserver.
(Assignee)

Comment 5

6 years ago
Comment on attachment 753900 [details] [diff] [review]
Patch v1

(In reply to Dão Gottwald [:dao] from comment #4)
> Comment on attachment 753900 [details] [diff] [review]
> Patch v1
> 
> You forgot to remove the bookmarksButton.tooltip string and
> bookmarksButtonObserver.

Ah yes, thank you.
Attachment #753900 - Flags: review?(jaws)
(Assignee)

Comment 6

6 years ago
Created attachment 753938 [details] [diff] [review]
Patch v1.1

Now with less bookmarksButton.tooltip and bookmarksButtonObserver.
Attachment #753900 - Attachment is obsolete: true
Attachment #753938 - Flags: review?(jaws)
Comment on attachment 753938 [details] [diff] [review]
Patch v1.1

Review of attachment 753938 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM!
Attachment #753938 - Flags: review?(jaws) → review+
(Assignee)

Comment 8

6 years ago
Landed on UX as https://hg.mozilla.org/projects/ux/rev/737b158fd80f
Whiteboard: [Australis:M6] → [Australis:M6][fixed-in-ux]

Comment 9

5 years ago
https://hg.mozilla.org/mozilla-central/rev/737b158fd80f
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [Australis:M6][fixed-in-ux] → [Australis:M6]
Target Milestone: --- → Firefox 28

Comment 10

5 years ago
mistake. please bring it back. it's a productivity thing and speeds up my work. sometimes I need to quickly click the sidebar on or off for a given wide page.
CTR add-on can add back the sidebar buttons, if you want to try it
https://addons.mozilla.org/firefox/addon/classicthemerestorer/
A much simpler add-on could easily be created, just for the purpose of adding back the sidebar add-ons, I don't know if anyone made one yet.
You need to log in before you can comment on or make changes to this bug.