Closed Bug 1477930 Opened 2 years ago Closed 2 years ago
46 bytes, text/x-phabricator-request
|Details | Review|
The hastags API is only used by the history result to decide whether to show the expander glyph close to the Tags entry in the Library. We may not be able to get a synchronous tags status in the future, we could maybe have a delayed cache through a trigger on the new tags table. Anyway, for now we can't do much more than removing this and accepting the glyph may be lying to the user; it will show the Tags section as open-able, but won't show children when opened. The only alternative would be to make it actually open the container, check for children and close it, but that would execute a synchronous query that'd be pointless work for most of the users.
The tagging API is moving to the bookmarking API, this is part of it. This will break the tags root open glyph in the Library, for now we don't have an alternative, we'll re-evaluate in the future in bug 1477934.
Comment on attachment 8994501 [details] Bug 1477930 - Remove taggingService HasTags synchronous API. r=standard8 Mark Banner (:standard8) has approved the revision. https://phabricator.services.mozilla.com/D2318
Attachment #8994501 - Flags: review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/285d7793a541 Remove taggingService HasTags synchronous API. r=Standard8
You need to log in before you can comment on or make changes to this bug.