Closed Bug 1393237 Opened 4 years ago Closed 4 years ago

Bookmark star has wrong tooltip after bookmarking a page

Categories

(Firefox :: General, defect, P1)

defect

Tracking

()

RESOLVED FIXED
Firefox 57
Tracking Status
firefox57 --- fixed
firefox59 --- verified

People

(Reporter: adw, Assigned: adw)

Details

Attachments

(1 file)

When you click the bookmark star in the urlbar to bookmark a page, its tooltip still reads "Bookmark this page" instead of "Edit this bookmark."  It looks like the problem is related to the star animation on click.  #star-button-animatable-image doesn't have a tooltip, and it's being shown on top of #star-button.  When you bookmark a page in a way that bypasses the star button (like the menu item), the problem doesn't happen.
Comment on attachment 8900486 [details]
Bug 1393237 - Bookmark star has wrong tooltip after bookmarking a page.

https://reviewboard.mozilla.org/r/171854/#review177108

I imagine we could clean up some CSS that uses `#star-button[starred] + #star-button-animatable-box > #star-button-animatable-image` to now just be `#star-button-animatable-image[starred]`. Do you want to do that while you're here?
Attachment #8900486 - Flags: review?(jaws) → review+
Sounds good, thanks for the quick review.
Summarizing a conversation between Jared and me on IRC:

All the relevant selectors look like this:

> #star-button-box[animationsenabled] > #star-button[starred][animate] + #star-button-animatable-box > #star-button-animatable-image

`animate` isn't set on the animatable-image, so it's not really possible to simplify these selectors more.
Pushed by dwillcoxon@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e7e15a0d5195
Bookmark star has wrong tooltip after bookmarking a page. r=jaws
https://hg.mozilla.org/mozilla-central/rev/e7e15a0d5195
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
QA Whiteboard: [good first verify]
I have reproduced this bug with 57.0a1 (2017-08-23) (64-bit) on Windows 7, 64 Bit!

This bug's fix is verified with latest Beta!

Build ID   : 20171016185129
User Agent : Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:57.0) Gecko/20100101 Firefox/57.0
QA Whiteboard: [good first verify] → [good first verify] [bugday-20171018]
I have Reproduced this bug with Nightly 59.0a1 (2017-12-10) on Windows 10, (64-bit)

Status: Fixed & Verified 

Useragent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:59.0) Gecko/20100101 Firefox/59.0 
BuildID: 20171210220040

[testday-20171208]
You need to log in before you can comment on or make changes to this bug.