fix: module identifier, don't merge modules with different media/suppors/layer (#1021)

Author: alexander-akait

src/index.js

@@ -165,7 +165,7 @@ class MiniCssExtractPlugin {
       }
 
       identifier() {
-        return `css|${this._identifier}|${this._identifierIndex}`;
+        return `css|${this._identifier}|${this._identifierIndex}|${this.layer}|${this.supports}|${this.media}}}`;
       }
 
       /**

test/cases/issue-css-loader-1503-1/dark.css

@@ -0,0 +1,3 @@
+body {
+  background: black;
+}

test/cases/issue-css-loader-1503-1/expected/dark.css

@@ -0,0 +1,4 @@
+body {
+  background: black;
+}
+

test/cases/issue-css-loader-1503-1/expected/index.css

@@ -0,0 +1,8 @@
+@supports (display: flex) {
+body {
+  background: black;
+}
+
+}
+
+

test/cases/issue-css-loader-1503-1/index.css

@@ -0,0 +1 @@
+@import url("dark.css") supports(display: flex);

test/cases/issue-css-loader-1503-1/webpack.config.js

@@ -0,0 +1,31 @@
+import Self from "../../../src";
+
+module.exports = {
+  entry: {
+    dark: "./dark.css",
+    index: "./index.css",
+  },
+  module: {
+    rules: [
+      {
+        test: /\.css$/,
+        use: [
+          {
+            loader: Self.loader,
+          },
+          {
+            loader: "css-loader",
+            options: {
+              modules: true,
+            },
+          },
+        ],
+      },
+    ],
+  },
+  plugins: [
+    new Self({
+      filename: "[name].css",
+    }),
+  ],
+};

test/cases/issue-css-loader-1503-2/dark.css

@@ -0,0 +1,3 @@
+body {
+  background: black;
+}

test/cases/issue-css-loader-1503-2/expected/dark.css

@@ -0,0 +1,4 @@
+body {
+  background: black;
+}
+

test/cases/issue-css-loader-1503-2/expected/index.css

@@ -0,0 +1,8 @@
+@layer framework.component {
+body {
+  background: black;
+}
+
+}
+
+

test/cases/issue-css-loader-1503-2/index.css

@@ -0,0 +1 @@
+@import url("dark.css") layer(framework.component);

test/cases/issue-css-loader-1503-2/webpack.config.js

@@ -0,0 +1,31 @@
+import Self from "../../../src";
+
+module.exports = {
+  entry: {
+    dark: "./dark.css",
+    index: "./index.css",
+  },
+  module: {
+    rules: [
+      {
+        test: /\.css$/,
+        use: [
+          {
+            loader: Self.loader,
+          },
+          {
+            loader: "css-loader",
+            options: {
+              modules: true,
+            },
+          },
+        ],
+      },
+    ],
+  },
+  plugins: [
+    new Self({
+      filename: "[name].css",
+    }),
+  ],
+};

test/cases/issue-css-loader-1503/dark.css

@@ -0,0 +1,3 @@
+body {
+  background: black;
+}

test/cases/issue-css-loader-1503/expected/dark.css

@@ -0,0 +1,4 @@
+body {
+  background: black;
+}
+

test/cases/issue-css-loader-1503/expected/index.css

@@ -0,0 +1,8 @@
+@media (prefers-color-scheme: dark) {
+body {
+  background: black;
+}
+
+}
+
+

test/cases/issue-css-loader-1503/index.css

@@ -0,0 +1 @@
+@import "./dark.css" (prefers-color-scheme: dark);

test/cases/issue-css-loader-1503/webpack.config.js

@@ -0,0 +1,31 @@
+import Self from "../../../src";
+
+module.exports = {
+  entry: {
+    dark: "./dark.css",
+    index: "./index.css",
+  },
+  module: {
+    rules: [
+      {
+        test: /\.css$/,
+        use: [
+          {
+            loader: Self.loader,
+          },
+          {
+            loader: "css-loader",
+            options: {
+              modules: true,
+            },
+          },
+        ],
+      },
+    ],
+  },
+  plugins: [
+    new Self({
+      filename: "[name].css",
+    }),
+  ],
+};