Page MenuHomePhabricator

Exception: .agora-field-styling is undefined: failed at `.agora-field-styling();`
Closed, ResolvedPublic

Description

  1. /srv/vagrant/mediawiki/extensions/UrlShortener/less/ext.urlShortener.special.less in the "ext.urlShortener.special.styles" module

Exception: .agora-field-styling is undefined: failed at .agora-field-styling(); /srv/vagrant/mediawiki/extensions/UrlShortener/less/ext.urlShortener.special.less on line 12

/srv/vagrant/mediawiki/includes/libs/lessc.inc.php:3581
/srv/vagrant/mediawiki/includes/libs/lessc.inc.php:2125
/srv/vagrant/mediawiki/includes/libs/lessc.inc.php:748
/srv/vagrant/mediawiki/includes/libs/lessc.inc.php:311
/srv/vagrant/mediawiki/includes/libs/lessc.inc.php:249
/srv/vagrant/mediawiki/includes/libs/lessc.inc.php:223
/srv/vagrant/mediawiki/includes/libs/lessc.inc.php:719
/srv/vagrant/mediawiki/includes/libs/lessc.inc.php:311
/srv/vagrant/mediawiki/includes/libs/lessc.inc.php:305
/srv/vagrant/mediawiki/includes/libs/lessc.inc.php:220
/srv/vagrant/mediawiki/includes/libs/lessc.inc.php:1927
/srv/vagrant/mediawiki/includes/libs/lessc.inc.php:1950
/srv/vagrant/mediawiki/tests/phpunit/LessFileCompilationTest.php:42
/srv/vagrant/mediawiki/tests/phpunit/MediaWikiTestCase.php:133
/srv/vagrant/mediawiki/tests/phpunit/MediaWikiPHPUnitCommand.php:42
/srv/vagrant/mediawiki/tests/phpunit/phpunit.php:160


Version: unspecified
Severity: normal

Details

Reference
bz68634

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:38 AM
bzimport set Reference to bz68634.
bzimport added a subscriber: Unknown Object (MLST).

Hmph, somebody used that? It was undocumented and not intended for public consumption, Jon and I removed it a few weeks ago.

Yeh. it's also styling this on 2 ids :-S

This is a good example of why mixins without parameter names are bad - this causes 3 sets of duplicate css.

The url shortner should be using mw-ui-button and mw-ui-input [1] when available.

[1] https://gerrit.wikimedia.org/r/149173

So, what's the migration plan? Undocumented or not, it ended up in the 1.23 release, so it got used (the extension is live on 2 wikifarms at least).

(In reply to Jon from comment #2)

The url shortner should be using mw-ui-button and mw-ui-input [1] when
available.

[1] https://gerrit.wikimedia.org/r/149173

Linked change hasn't been merged yet, so not really useful...

This line of css should be removed. I'm not sure why it was using that mixin in the first place.

I suspect it was trying to style an input in the mediawiki ui style.
If it wants to use that style it will need to wait for that patch to be merged so help getting it merged appreciated.

Is that UrlShortener using a CSS definition that get removed from core?

Change 149902 had a related patch set uploaded by Jdlrobson:
Fix LESS exception

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

Too much talking :P
Above fixes it.
There are follow ups which can be merged without dependencies that fix the styling.

Change 149902 merged by Bartosz Dziewoński:
Fix LESS exception

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