Closed Bug 1393237 Opened 6 years ago Closed 6 years ago
Bookmark star has wrong tooltip after bookmarking a page
59 bytes, text/x-review-board-request
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 email@example.com: https://hg.mozilla.org/integration/autoland/rev/e7e15a0d5195 Bookmark star has wrong tooltip after bookmarking a page. r=jaws
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.