Page MenuHomePhabricator

Add specific class to Score error box
Closed, ResolvedPublic

Description

Until T257066 is fixed, projects may want to hide Score’s error boxes (e.g. request on frwiki). Currently such CSS rules can only target .errorbox, which may accidentally hide other error boxes, no matter how specific they are. There should be a specific class like .mw-ext-score-error as well, which can be safely targeted with on-site CSS, and ideally this should be deployed ASAP so that projects don’t create vague CSS rules.

Event Timeline

Change 610967 had a related patch set uploaded (by Reedy; owner: Reedy):
[mediawiki/extensions/Score@master] Make Score errors use a specific css class

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

Change 610967 merged by jenkins-bot:
[mediawiki/extensions/Score@master] Make Score errors use a specific css class

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

@Reedy Thanks for the patch! Is there any chance to get this deployed today? I know WMF employees want to enjoy the weekend, but volunteers enjoy the weekend by editing Wikipedia and might create inferior CSS rules to work around this issue before Monday.

Change 611217 had a related patch set uploaded (by Reedy; owner: Reedy):
[mediawiki/extensions/Score@wmf/1.35.0-wmf.40] Make Score errors use a specific css class

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

Change 611217 merged by Reedy:
[mediawiki/extensions/Score@wmf/1.35.0-wmf.40] Make Score errors use a specific css class

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

Reedy claimed this task.

@Reedy Thanks for the patch! Is there any chance to get this deployed today? I know WMF employees want to enjoy the weekend, but volunteers enjoy the weekend by editing Wikipedia and might create inferior CSS rules to work around this issue before Monday.

Done!

@Reedy Thanks for the patch! Is there any chance to get this deployed today? I know WMF employees want to enjoy the weekend, but volunteers enjoy the weekend by editing Wikipedia and might create inferior CSS rules to work around this issue before Monday.

Done!

Thanks!

Change 615766 had a related patch set uploaded (by Reedy; owner: Reedy):
[mediawiki/extensions/Score@REL1_34] Make Score errors use a specific css class

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

Change 615767 had a related patch set uploaded (by Reedy; owner: Reedy):
[mediawiki/extensions/Score@REL1_31] Make Score errors use a specific css class

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

Change 615767 merged by Reedy:
[mediawiki/extensions/Score@REL1_31] Make Score errors use a specific css class

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

Change 615766 merged by jenkins-bot:
[mediawiki/extensions/Score@REL1_34] Make Score errors use a specific css class

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