Closed Bug 1768004 Opened 2 months ago Closed 2 months ago

Remove proton places tooltip code.

Categories

(Toolkit :: Themes, defect)

defect

Tracking

()

RESOLVED FIXED
102 Branch
Tracking Status
firefox102 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

Details

Attachments

(1 file)

There's nobody working on it, and tooltips should hopefully be nice enough after recent changes. Having it enabled causes artifacts like bug 1767815 comment 3.

There's nobody working on it, and tooltips should hopefully be nice
enough after recent changes (bug 1765423).

Having it enabled causes artifacts like bug 1767815 comment 3. We can
always rescue this from hg history if needed.

Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/089ee3fa51ea
Remove proton places tooltip code. r=dao
Pushed by emilio@crisal.io:
https://hg.mozilla.org/integration/autoland/rev/86beb106f884
Remove some leftover usage I missed and ESLint caught.
Pushed by emilio@crisal.io:
https://hg.mozilla.org/integration/autoland/rev/7794039ba4da
Remove another unused variable.
Status: NEW → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 102 Branch

Isn't bhTooltip still a places-tooltip? Seems basically the same minus the lower hbox. But I would be willing to maintain the tab tooltip, I rather liked the ability to view the URL. It could be formalized better so there's less redundancy in the label setting.

You need to log in before you can comment on or make changes to this bug.