Closed Bug 1591790 Opened 2 years ago Closed 2 months ago

Return bookmark paths in PlacesUtils.bookmarks.fetch

Categories

(Toolkit :: Places, enhancement, P3)

All
Unspecified
enhancement

Tracking

()

RESOLVED FIXED
90 Branch
Tracking Status
firefox90 --- fixed

People

(Reporter: fluks.github, Assigned: lebar)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

PlacesUtils.bookmarks.fetch should also return paths of a bookmark to make it possible to implement a feature in Library window, and maybe in bookmarks sidebar too, to open enclosing folder of a bookmark.

See also this comment.

yes, the idea was to have an includePath option added to the options object, and if it's set the query can use RECURSIVE to fetch the path, that may be an array of { title, guid} objects, probably starting from the root and arriving to the parent.

Priority: -- → P3

Any progress on this?

(In reply to Robert Townley from comment #2)

Any progress on this?

Generally you should be able to assume that if there's been no comments on a bug recently (and nothing in any attachments), then there's been no progress.

Assignee: nobody → lebar
Status: NEW → ASSIGNED

Depends on D110786

Atm we're a bit overloaded with the Proton work, but I'll try to look at this soon. I added Mark as a first-come-first-serve reviewer, though he's out of office this week... Anyway, thank you for you contribution and hopefully we can take it very soon.

Attachment #9213519 - Attachment description: Bug 1591790 - Return bookmark paths in fetch. r?mak → Bug 1591790 - Return bookmark paths in fetch. r=mak,Standard8
Attachment #9213520 - Attachment is obsolete: true
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f897c6bb3c38
Return bookmark paths in fetch. r=Standard8
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
You need to log in before you can comment on or make changes to this bug.