Bookmark Menu object on the customize screen needs to be renamed "Bookmarks Menu"

VERIFIED FIXED in Firefox 57

Status

()

enhancement
P1
normal
VERIFIED FIXED
2 years ago
2 years ago

People

(Reporter: bbell, Assigned: Gijs)

Tracking

57 Branch
Firefox 57
Points:
---

Firefox Tracking Flags

(firefox57 fixed)

Details

(Whiteboard: [reserve-photon-structure][photon-l10n-risk])

Attachments

(3 attachments)

This name change is necessary because The Bookmark Menu is referenced as the "Bookmark Menu" in the Library where we offer users the option to "Open Bookmark Menu," and as a destination users can save bookmarks to in the save dialog where users can "Save to Bookmark Menu." However, you can't find a "Bookmark Menu" under customize, because it's labeled "Bookmarks."
Whiteboard: [photon-structure]
Posted image detail.png
Screenshot of what needs to be renamed.
Component: Menus → Bookmarks & History
Component: Bookmarks & History → Menus
Summary: Bookmark Menu object on the customize screen needs to be renamed "Bookmark Menu" → Bookmark Menu object on the customize screen needs to be renamed "Bookmarks Menu"
(In reply to bbell from comment #0)
> This name change is necessary because The Bookmark Menu is referenced as the
> "Bookmark Menu" in the Library where we offer users the option to "Open
> Bookmark Menu," and as a destination users can save bookmarks to in the save
> dialog where users can "Save to Bookmark Menu." However, you can't find a
> "Bookmark Menu" under customize, because it's labeled "Bookmarks."

"Bookmarks Menu" (plural) should be the label.
Whiteboard: [photon-structure] → [photon-structure] [triage]
Priority: -- → P4
Whiteboard: [photon-structure] [triage] → [reserve-photon-structure]
Whiteboard: [reserve-photon-structure] → [reserve-photon-structure][photon-l10n-risk]
Taking this because of the strings involved.

One question I have here: when you put this button in the overflow panel (either pin it or when it appears there because there's no space on the toolbar), it will show the same bookmarks view as the library does (same items/content, same title).

There is also normally only 1 label for the button that gets used both in customize mode and in the panel. The labeling situation right now is:

palette: "Bookmarks"
panel: "Bookmarks"
title of the subview in panel: "Bookmarks"
title of the subview in library panel: "Bookmarks"


The situation after the patches I've just attached is:

palette: "Bookmarks Menu"
panel: "Bookmarks Menu"
title of the subview in panel: "Bookmarks"
title of the subview in library panel: "Bookmarks"


This is a bit confusing because you'd expect the thing in the panel to show you the "bookmarks menu" and it doesn't (though it will if you move it to any of the toolbars)

In principle, I could write code to make the label for the customize mode palette and the panel different, so it changes label when you move it between these items, but that also seems confusing.

Really, I don't know how to weigh this inconsistency with the one from comment #0. I think we're a bit out of time to implement a separate bookmarks menu view for the bookmarks menu button to use when it's in the overflow panel... (ie that opens any folders as subviews or by opening the library, the latter being what we did pre-photon... I guess we could try to resurrect that code from hg history... but it sounds hairy.)

Bryan, what would you like to do? Take this patch, keep the status quo, or something else?
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Iteration: --- → 57.2 - Aug 29
Flags: needinfo?(bbell)
Flags: qe-verify?
Priority: P4 → P1
Comment on attachment 8900707 [details]
Bug 1389625 - change bookmarks menu button label,

https://reviewboard.mozilla.org/r/172154/#review177676
Attachment #8900707 - Flags: review?(adw) → review+
Comment on attachment 8900708 [details]
Bug 1389625 - clean up subview headers now that we use the title of the view instead,

https://reviewboard.mozilla.org/r/172156/#review177678
Attachment #8900708 - Flags: review?(adw) → review+
Discussed with Bryan, we're gonna go ahead with the rename.
Flags: needinfo?(bbell)
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/1707066b3257
change bookmarks menu button label, r=adw
https://hg.mozilla.org/integration/autoland/rev/266a47a36a77
clean up subview headers now that we use the title of the view instead, r=adw
https://hg.mozilla.org/mozilla-central/rev/1707066b3257
https://hg.mozilla.org/mozilla-central/rev/266a47a36a77
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
I have reproduced this bug with Nightly 57.0a1 (2017-08-11) on Windows 8.1, 64 Bit!

This bug's fix is verified on Latest Nightly 57.0a1

Build ID : 20170830100404
User Agent : Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:57.0) Gecko/20100101 Firefox/57.0
QA Whiteboard: [bugday-20170830]
Status: RESOLVED → VERIFIED
Flags: qe-verify?
You need to log in before you can comment on or make changes to this bug.