Last modified: 2014-05-29 17:43:22 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T67434, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 65434 - Weird watchlist star animation on Firefox
Weird watchlist star animation on Firefox
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
unspecified
All All
: High minor (vote)
: 1.23.0 release
Assigned To: Bartosz Dziewoński
: code-update-regression
: 65853 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-05-17 12:39 UTC by Helder
Modified: 2014-05-29 17:43 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---
matma.rex: Backport_to_Stable+


Attachments
Screencast (483.95 KB, video/ogg)
2014-05-17 12:39 UTC, Helder
Details

Description Helder 2014-05-17 12:39:03 UTC
Created attachment 15425 [details]
Screencast

I don't think the borders should be animated together with the star icon. This is happening on Firefox 29. It is fine on Google Chrome 34.
Comment 1 Bartosz Dziewoński 2014-05-17 13:07:35 UTC
The borders look like Firefox's styling for active or focused <a> elements, the watch star is an <a> element, and we apply a rotation animation to it when clicked.

Also, it looks hilarious.

This might be a regression from Krinkle's 678b4d554 / bug 64934.
Comment 2 Bartosz Dziewoński 2014-05-17 13:32:07 UTC
Assuming we want to keep the outline normally, the simplest fix for this would be '#ca-unwatch.icon a.loading, #ca-watch.icon a.loading { outline: none; }'. The watch star shouldn't really be clickable during the animation anyway (although it currently is, which is a separate bug).
Comment 3 Gerrit Notification Bot 2014-05-17 14:02:07 UTC
Change 133894 had a related patch set uploaded by Bartosz Dziewoński:
vector: Suppress watch star focus outline when animating it

https://gerrit.wikimedia.org/r/133894
Comment 4 Gerrit Notification Bot 2014-05-17 14:02:12 UTC
Change 133895 had a related patch set uploaded by Bartosz Dziewoński:
mediawiki.page.watch.ajax: Disable watch link while waiting for API response

https://gerrit.wikimedia.org/r/133895
Comment 5 Gerrit Notification Bot 2014-05-18 10:03:09 UTC
Change 133894 merged by jenkins-bot:
vector: Suppress watch star focus outline when animating it

https://gerrit.wikimedia.org/r/133894
Comment 6 Gerrit Notification Bot 2014-05-19 10:59:43 UTC
Change 134070 had a related patch set uploaded by Bartosz Dziewoński:
vector: Suppress watch star focus outline when animating it

https://gerrit.wikimedia.org/r/134070
Comment 7 Gerrit Notification Bot 2014-05-19 11:05:23 UTC
Change 134070 merged by jenkins-bot:
vector: Suppress watch star focus outline when animating it

https://gerrit.wikimedia.org/r/134070
Comment 8 Liangent 2014-05-28 14:05:44 UTC
*** Bug 65853 has been marked as a duplicate of this bug. ***
Comment 9 Liangent 2014-05-28 18:56:42 UTC
bug 64934 has been backported?
Comment 10 Bartosz Dziewoński 2014-05-29 17:43:22 UTC
This was backported to 1.23 too, https://gerrit.wikimedia.org/r/#/c/134070/.

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


Navigation
Links