Unsorted Bookmarks entry in menu of Bookmarks button in Personal Toolbar has no special icon

RESOLVED FIXED in seamonkey2.1b1

Status

SeaMonkey
Bookmarks & History
--
minor
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

Tracking

Trunk
seamonkey2.1b1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
Now with Places-based bookmarks, the Unsorted Bookmarks entry in the Bookmarks Manager has a special icon, but the same entry in the menu of the Bookmarks button in the Personal Toolbar only shows the standard folder icon.
(Assignee)

Comment 1

7 years ago
Created attachment 466170 [details] [diff] [review]
add icon definition [Checkin: comment 10]

AFAICS the icon definition for Unsorted Bookmarks inside the Bookmarks Manager is #BMB_unsortedBookmarksFolderMenu:
http://mxr.mozilla.org/comm-central/source/mozilla/browser/themes/winstripe/browser/browser.css#1654

OTOH, our icon definition for Personal Toolbar inside the drop-down menu of the Bookmarks button is #BMB_bookmarksToolbarFolderMenu:
http://mxr.mozilla.org/comm-central/source/suite/themes/classic/communicator/bookmarks/bookmarks.css#105

In the latter file there is already a line for #unsortedBookmarksFolderMenu which is just missing the extra #BMB_bookmarksToolbarFolderMenu.

Requesting 2.1a3 approval for simple and safe fix for a new feature (UX polish).

BTW: Do we have a bug somewhere to find a common name for "Bookmarks Toolbar" in the Bookmarks Manager vs. "Personal Toolbar" in the button drop-down menu?
Assignee: nobody → jh
Status: NEW → ASSIGNED
Attachment #466170 - Flags: superreview?(neil)
Attachment #466170 - Flags: review?(neil)
Attachment #466170 - Flags: approval-seamonkey2.1a3?

Comment 2

7 years ago
Comment on attachment 466170 [details] [diff] [review]
add icon definition [Checkin: comment 10]

Too late for a plus patch has not been reviewed yet.
Attachment #466170 - Flags: approval-seamonkey2.1a3?

Comment 3

7 years ago
Why do we only have Unsorted Bookmarks in the Bookmarks button and not the menu?
(Assignee)

Comment 4

7 years ago
(In reply to comment #3)
> Why do we only have Unsorted Bookmarks in the Bookmarks button and not the
> menu?

[Just in case anyone thought I could answer that: ] Sorry, no idea, I didn't follow the Places-Bookmarks implementation that closely. Would be another bug anyway.
(Assignee)

Comment 5

7 years ago
(In reply to comment #1)
> BTW: Do we have a bug somewhere to find a common name for "Bookmarks Toolbar"
> in the Bookmarks Manager vs. "Personal Toolbar" in the button drop-down menu?

Filed bug 588191.

KaiRo, can you enlighten us why Unsorted Bookmarks is missing from the menu as Neil noted? Should we file a bug for that as well?

Comment 6

7 years ago
(In reply to comment #5)
> KaiRo, can you enlighten us why Unsorted Bookmarks is missing from the menu as
> Neil noted? Should we file a bug for that as well?

I probably just copied some logic from Firefox bookmarks toolbar button and bookmarks menu. I'm not sure if it's really needed/wanted in either on our side or none, but you are right that we probably should make it consistent.
(Assignee)

Comment 7

7 years ago
(In reply to comment #6)
> I'm not sure if it's really needed/wanted in either on our side
> or none, but you are right that we probably should make it consistent.

Filed bug 588807.

Comment 8

7 years ago
Comment on attachment 466170 [details] [diff] [review]
add icon definition [Checkin: comment 10]

Sorry for the delay on this.

There's no reason for us to penalise keyboard users; we should totally have the unsorted bookmarks folder accessible from the main menu too.
Attachment #466170 - Flags: superreview?(neil)
Attachment #466170 - Flags: superreview+
Attachment #466170 - Flags: review?(neil)
Attachment #466170 - Flags: review+
(Assignee)

Comment 9

7 years ago
(In reply to comment #8)
> There's no reason for us to penalise keyboard users; we should totally have the
> unsorted bookmarks folder accessible from the main menu too.

I assume that you confused this bug with bug 588807 (see comment 2 there) when you wrote that but that the reviews still apply to this patch, right? ;-)
(Assignee)

Comment 10

7 years ago
Comment on attachment 466170 [details] [diff] [review]
add icon definition [Checkin: comment 10]

http://hg.mozilla.org/comm-central/rev/c2aa1b4c38d6
Attachment #466170 - Attachment description: add icon definition → add icon definition [Checkin: comment 10]
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b1
You need to log in before you can comment on or make changes to this bug.