(In reply to fluks from comment #42)
> (In reply to Marco Bonardo [::mak] from comment #41)
> > I'd be ok extending PlacesUtils.bookmarks.fetch to also fetch the path of a 
> > bookmark if a specific option is passed into it (so, on demand).
> 
> Do you mean having an about:config entry?

no, a property of the options object that is passed to fetch()
I was thinking about something like `let bm = await 
PlacesUtils.bookmarks.fetch(guid, undefined, { includePath: true });` and you'd 
get back a bookmark object that includes a `path` property, that may be 
something like `[ {guid, title}, {guid, title}, ... ]` starting from the parent 
up to the first meaningful root (toolbar, menu, unfiled, mobile, ...).

You could start filing a dependency bug in toolkit /Places to add that,
then here you'd use that API to implement the feature in browser.

-- 
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