Created attachment 9237625
menu_proposal.png

(In reply to Mark Banner (:standard8) from comment #62)
> Adding needinfo to Romain as I think that was missed. Note: this is a patch 
> from a contributor for a long standing paper-cut issue, and we mainly just 
> need help with an appropriate string.

Thanks for the ping, first I want to make sure that I correctly understand the 
change, my understanding is:
1 Open the library modal
2 Enter a search term in the "Search bookmarks" search bar, results appear
3 Right click one of the search results, context menu appears. One new entry 
"Open Containing Folder" is available underneath the other "Open" entries
4 Selecting  "Open Containing Folder" opens the bookmark folder and selects it 
on the left hand side and selects the bookmark on the right hand side

Assuming the above is right I'd propose the following:
- use "Open Containing Folder" for consistency with the "Downloads" section of 
the "Library" Window where this context menu entry already exists
- Locate this new entry underneath other "Open" entries

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1672139

Title:
  Add "Open Enclosing Folder" context menu to search results of
  bookmarks in the Library window (link to view/open containing folder,
  parent folder button)

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/1672139/+subscriptions


-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to