Page MenuHomePhabricator

[css-3.less test case] Add support for /deep/ selector
Closed, ResolvedPublic

Event Timeline

Change 1003506 had a related patch set uploaded (by Hokwelum; author: Hokwelum):

[mediawiki/libs/less.php@master] Add support for deep Selector

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

Change 1003506 merged by jenkins-bot:

[mediawiki/libs/less.php@master] Add support for /deep/ selectors

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

One thing remaining here, is a follow-up commit that removes the old _outputMap.

Change 1004661 had a related patch set uploaded (by Hokwelum; author: Hokwelum):

[mediawiki/libs/less.php@master] Clean up all reference to $_outputMap

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

Change 1004661 merged by jenkins-bot:

[mediawiki/libs/less.php@master] Remove references to Less_Environment::$_outputMap

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

pmiazga subscribed.

This task is supposed to fix the import-reference fixture, but this test is still failing:

--- actual
+++ /Users/pmiazga/projects/wmf/less.php/test/Fixtures/lessjs-2.5.3/css/import-reference.css
@@ -1,4 +1,3 @@
-@import url("import/test.css");
 input[type="text"].class#id[attr=32]:not(1) {
   color: white;
 }
@@ -35,8 +34,14 @@
 .visible + .visible .sub {
   color: green;
 }
-this isn't very valid CSS.
-this isn't very valid CSS.
+@supports (something: else) {
+  .class {
+    something: else;
+  }
+  .nestedToo .class {
+    something: else;
+  }
+}
 .b {
   color: red;
   color: green;
@@ -54,7 +59,7 @@
   color: green;
 }
 .y {
-  pulled-in: yes;
+  pulled-in: yes /* inline comment survives */;
 }
 /* comment pulled in */
 .visible {
@@ -72,6 +77,20 @@
 .test {
   color: red;
 }
+.test:first-child {
+  color: blue;
+}
+@keyframes some-name {
+  property: value;
+}
+@supports (animation-name: test) {
+  @keyframes some-name {
+    property: value;
+  }
+  .selector {
+    color: red;
+  }
+}
 div {
   this isn't very valid CSS.
 }

I'm reopening this ticket, after fixing the issue please remember to remove the

			'import-reference' => true, // T352862

from known failing fixtures (https://gerrit.wikimedia.org/g/mediawiki/libs/less.php/+/9105597e08185b135748d9f6968d5359dbd78783/test/phpunit/FixturesTest.php#34)

@pmiazga I have created a task here T362647 to investigate the CSS difference in import-reference test. Looks like it's no longer failing because of the deep selector :-)

I'll update import-reference to point to that task in the current patch I'm working on!

Krinkle renamed this task from Add support for /deep/ selector (css-3 test case) to [css-3.less test case] Add support for /deep/ selector.Apr 16 2024, 1:43 PM