Merge pull request #6 from yandex/fix_empty_comments_at_rule_creation

Fix empty comments at rule copy

packages/postcss-logical-fallback/package.json

@@ -1,6 +1,6 @@
 {
   "name": "postcss-logical-fallback",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "PostCSS plugin for logical fallback props",
   "main": "index.js",
   "files": [

packages/postcss-logical-fallback/src/utils/has-empty-child-nodes.ts

@@ -1,3 +1,6 @@
-import {Container} from "postcss";
+import { Container } from 'postcss';
 
-export const hasEmptyChildNodes = ({nodes}: Container) => !nodes || nodes.length === 0
+export const hasEmptyChildNodes = ({ nodes }: Container) => {
+    const significantNodes = nodes.filter((node) => node.type !== 'comment');
+    return !significantNodes || significantNodes.length === 0;
+};

packages/postcss-logical-fallback/tests/no-changes.test.ts

@@ -0,0 +1,38 @@
+import { test } from 'uvu';
+import { run } from './run';
+
+test('should not copy comments', async () => {
+    await run(
+        `body {
+      /* stylelint-disable-next-line plugin/no-unsupported-browser-features */
+      min-height: 100vh;
+      padding: 2rem;
+  }`,
+        `body {
+      /* stylelint-disable-next-line plugin/no-unsupported-browser-features */
+      min-height: 100vh;
+      padding: 2rem;
+  }`,
+    );
+
+    await run(
+        `.class {
+      /*
+        multiline
+        comment
+       */
+      min-height: 100vh;
+      padding: 2rem;
+  }`,
+        `.class {
+      /*
+        multiline
+        comment
+       */
+      min-height: 100vh;
+      padding: 2rem;
+  }`,
+    );
+});
+
+test.run();