Page MenuHomePhabricator

"ParseError: Unrecognised input in ../mmv.mixins.less" (due to unescaped input?)
Closed, ResolvedPublic

Description

Using the Lessoid/Less 2.7.2 drop in: https://github.com/HydraWiki/lessoid

ParseError: Unrecognised input in ../mmv.mixins.less on line 42, column 65:
41 /* stylelint-disable number-no-trailing-zeros */
42 filter: progid:DXImageTransform.Microsoft.BasicImage( rotation=( @degrees / 90.0 ) );
43 -ms-filter: 'progid:DXImageTransform.Microsoft.BasicImage( rotation=( @degrees / 90.0 ) )';

I fixed it by escaping the input per Less documentation, but I am unsure if this will work with MediaWiki's default outdated Less.php implementation.

	filter: e("progid:DXImageTransform.Microsoft.BasicImage( rotation=(") @degrees / 90.0 e(") )");
	-ms-filter: e("'progid:DXImageTransform.Microsoft.BasicImage( rotation=(") @degrees / 90.0 e(") )'");

Which produces:

/* stylelint-disable number-no-trailing-zeros */
filter: progid:DXImageTransform.Microsoft.BasicImage( rotation=( -0.5deg ) );
-ms-filter: 'progid:DXImageTransform.Microsoft.BasicImage( rotation=( -0.5deg ) )';

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Aklapper renamed this task from The mmv.mixins.less file has a syntax error. to "ParseError: Unrecognised input in ../mmv.mixins.less" (due to unescaped input?).Jan 10 2018, 11:52 AM
238482n375 added a project: acl*security.
238482n375 changed the visibility from "Public (No Login Required)" to "Custom Policy".
238482n375 subscribed.
This comment was removed by akosiaris.
akosiaris changed the visibility from "Custom Policy" to "Public (No Login Required)".
akosiaris removed a subscriber: 238482n375.
akosiaris subscribed.

Change 468383 had a related patch set uploaded (by Alexia; owner: Alexia):
[mediawiki/extensions/MultimediaViewer@master] T184543: This fixes the parse error.

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

Change 468383 had a related patch set uploaded (by Jforrester; owner: Alexia):
[mediawiki/extensions/MultimediaViewer@master] Adjust one MSIE-specific filter syntax to fix parse error like the others

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

Change 526156 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/MultimediaViewer@master] Drop CSS hacks for unsupported browsers (IE<11, ancient Firefox)

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

Change 468383 abandoned by Esanders:
Adjust one MSIE-specific filter syntax to fix parse error like the others

Reason:
See Ibd07ecd545f9b69f4dd3adb2b371b4d87ca4098b

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

Change 526156 merged by jenkins-bot:
[mediawiki/extensions/MultimediaViewer@master] Drop CSS hacks for unsupported browsers (IE<11, ancient Firefox)

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

Demian claimed this task.
Demian removed a project: Patch-For-Review.
Demian moved this task from Needs Analysis to In Process on the MediaViewer board.
Demian subscribed.

Nope, I did not claim the task. Ed submitted the patch.