mediawiki.ui: dependency to module .anchor caused button regression
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Florian
	Sep 13 2014, 11:26 PM

Description

regression in font of button (hover)

I open this bug for Echo, because i haven't found any use for mediawiki.ui.anchor, so i want to ask the dev's if it's possible to remove this dependency from ext.echo.overlay:
https://github.com/wikimedia/mediawiki-extensions-Echo/blob/acce7cc9fbd093ab1d6149df7156ed77837cbe9c/Resources.php#L60

because it's unused (?) but caused a regression to buttons :hover and :focus color (see pictures) when wgUseMediaWikiUIEverywhere is true.

Version: master
Severity: normal
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=70801

Attached:

Details

Reference: bz70818

	Subject	Repo	Branch	Lines +/-
	Use mw-ui-anchor class	mediawiki/extensions/Echo	master	+1 -1

Customize query in gerrit

Revisions and Commits

rMEXT MediaWiki Extensions
	rMEXTd10b54543135 Updated mediawiki/extensions Project: mediawiki/extensions/Echo…
rECHO extension-Echo
	rECHOb930dfaf47fc Use mw-ui-anchor class

Related Objects
Search...

Status	Assigned	Task
Declined	• Mattflaschen-WMF	T99211 Eliminate $wgUseMediaWikiUIEverywhere, leaving 'on' behavior
Declined	• Mattflaschen-WMF	T72424 Set $wgUseMediaWikiUIEverywhere = true
Resolved	• Mattflaschen-WMF	T72818 mediawiki.ui: dependency to module .anchor caused button regression

Event Timeline

• bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:59 AM

• bzimport added a project: Notifications.

• bzimport set Reference to bz70818.

• bzimport added a subscriber: Unknown Object (MLST).

Florian created this task.Sep 13 2014, 11:26 PM

Created attachment 16463
regression in font of button (focus)

Attached:

It's not unused, unfortunately (it's used for the "Alerts (0)" or however many you have, which is "(echo-notification-alert)"), but this is a valid bug.

shahyar+wmfbugzilla wrote:

Should be resolved by https://gerrit.wikimedia.org/r/#/c/123804/

(In reply to Shahyar Ghobadpour from comment #3)

Should be resolved by https://gerrit.wikimedia.org/r/#/c/123804/

Tested -> solved. Thanks!

Patch was reverted.

• werdna unsubscribed.Dec 10 2014, 5:02 PM

• Mattflaschen-WMF claimed this task.Mar 2 2015, 6:58 PM

Change 194012 had a related patch set uploaded (by Mattflaschen):
Use mw-ui-anchor class

https://gerrit.wikimedia.org/r/194012

gerritbot added a project: Patch-For-Review.Mar 3 2015, 1:18 AM

The bug should not occur anymore since I merged rMW2568fbbdd42e: Change mw-ui anchors to require mw-ui-anchor base class, and introduce mw-ui…. However, that also means we weren't getting the desired color, since a class is now required. The patch solves that.

The core patch I linked prevents mw-ui-progressive from spilling over into other modules (i.e. affecting buttons) when no other class (e.g. mw-ui-text, mw-ui-anchor) is specified.

EBernhardson triaged this task as Medium priority.Mar 3 2015, 6:50 PM

EBernhardson subscribed.

Change 194012 merged by jenkins-bot:
Use mw-ui-anchor class

https://gerrit.wikimedia.org/r/194012

• Mattflaschen-WMF closed this task as Resolved by committing rECHOb930dfaf47fc: Use mw-ui-anchor class.Mar 6 2015, 1:03 AM

• Mattflaschen-WMF added a commit: rECHOb930dfaf47fc: Use mw-ui-anchor class.

Diffusion added a commit: rMEXTd10b54543135: Updated mediawiki/extensions Project: mediawiki/extensions/Echo….Mar 11 2015, 4:40 AM