postcss-nesting : spec compliance

plugin-packs/postcss-preset-env/src/lib/get-options-for-browsers-by-feature.mjs

@@ -1,6 +1,3 @@
-import browserslist from 'browserslist';
-import { getUnsupportedBrowsersByFeature } from './get-unsupported-browsers-by-feature.mjs';
-
 // add extra options for certain browsers by feature
 export default function getOptionsForBrowsersByFeature(supportedBrowsers, feature, cssdbList, options, logger) {
 	switch (feature.id) {
@@ -10,25 +7,6 @@ export default function getOptionsForBrowsersByFeature(supportedBrowsers, featur
 			return {
 				onComplexSelector: 'warning',
 			};
-		case 'nesting-rules':
-			{
-				// TODO : remove this in a next major release
-
-				// Nesting rules can transform selectors to use :is pseudo.
-				// This is more spec compliant but it's not supported by all browsers.
-				// If we can't use :is pseudo according to preset-env options, we add an extra option to avoid :is pseudo.
-				const feature = cssdbList.find(feature => feature.id === 'is-pseudo-class');
-
-				if (needsOptionFor(feature, supportedBrowsers)) {
-					logger.log('Disabling :is on "nesting-rules" due to lack of browser support.');
-					return {
-						noIsPseudoSelector: true,
-					};
-				}
-			}
-
-			return {};
-
 		case 'any-link-pseudo-class':
 			{
 				const hasIEOrEdge = supportedBrowsers.find((browserslistEntry) => {
@@ -64,18 +42,3 @@ export default function getOptionsForBrowsersByFeature(supportedBrowsers, featur
 			return {};
 	}
 }
-
-function needsOptionFor(feature, supportedBrowsers) {
-	const unsupportedIn = getUnsupportedBrowsersByFeature(feature);
-	const unsupportedBrowsers = browserslist(unsupportedIn, {
-		ignoreUnknownVersions: true,
-	});
-
-	if (supportedBrowsers.some((supportedBrowser) => {
-		return unsupportedBrowsers.some(polyfillBrowser => polyfillBrowser === supportedBrowser);
-	})) {
-		return true;
-	}
-
-	return false;
-}

plugin-packs/postcss-preset-env/test/_tape.mjs

@@ -64,18 +64,6 @@ postcssTape(plugin)({
 			browsers: 'op_mini all',
 		},
 	},
-	'basic:ch88-ff78:no-is-pseudo': {
-		message: ':is pseudo for nesting can be disable with modern browsers { browsers: "chrome >= 88, firefox >= 78", stage: 0, features: { nesting-rules: { noIsPseudoSelector: true } } } usage',
-		options: {
-			browsers: 'chrome >= 88, firefox >= 78',
-			stage: 0,
-			features: {
-				'nesting-rules': {
-					noIsPseudoSelector: true,
-				},
-			},
-		},
-	},
 	'basic:ch88-ff78-saf10': {
 		message: 'does not use :is pseudo for nesting with an older browser { browsers: "chrome >= 88, firefox >= 78, safari >= 10", stage: 0 } usage',
 		options: {
@@ -142,17 +130,6 @@ postcssTape(plugin)({
 			},
 		},
 	},
-	'basic:nesting:noIsPseudoSelector:false': {
-		message: 'supports { stage: false, features: { "nesting-rules": { "noIsPseudoSelector": false } } } usage',
-		options: {
-			stage: false,
-			features: {
-				'nesting-rules': {
-					noIsPseudoSelector: false,
-				},
-			},
-		},
-	},
 	'basic:nesting:false': {
 		message: 'supports { stage: 0, features: { "nesting-rules": false } } usage',
 		options: {

plugin-packs/postcss-preset-env/test/basic.autoprefixer.expect.css

@@ -58,7 +58,7 @@
 	order: 9;
 }
 
-.test-nesting-rules + p, #test-is-pseudo + p {
+.test-nesting-rules:not(#does-not-exist) + p,#test-is-pseudo + p {
 		order: 8;
 	}
 

plugin-packs/postcss-preset-env/test/basic.autoprefixer.false.expect.css

@@ -58,7 +58,7 @@
 	order: 9;
 }
 
-.test-nesting-rules + p, #test-is-pseudo + p {
+.test-nesting-rules:not(#does-not-exist) + p,#test-is-pseudo + p {
 		order: 8;
 	}
 

plugin-packs/postcss-preset-env/test/basic.autoprefixer.remove.false.expect.css

@@ -59,7 +59,7 @@
 	order: 9;
 }
 
-.test-nesting-rules + p, #test-is-pseudo + p {
+.test-nesting-rules:not(#does-not-exist) + p,#test-is-pseudo + p {
 		order: 8;
 	}
 

plugin-packs/postcss-preset-env/test/basic.ch38.expect.css

@@ -51,7 +51,7 @@
 	order: 9;
 }
 
-.test-nesting-rules + p, #test-is-pseudo + p {
+.test-nesting-rules:not(#does-not-exist) + p,#test-is-pseudo + p {
 		order: 8;
 	}
 

plugin-packs/postcss-preset-env/test/basic.ch88-ff78-saf10.expect.css

@@ -58,7 +58,7 @@
 	order: 9;
 }
 
-.test-nesting-rules + p, #test-is-pseudo + p {
+.test-nesting-rules:not(#does-not-exist) + p,#test-is-pseudo + p {
 		order: 8;
 	}
 

plugin-packs/postcss-preset-env/test/basic.ch88-ff78.expect.css

@@ -58,7 +58,7 @@
 	order: 9;
 }
 
-.test-nesting-rules + p, #test-is-pseudo + p {
+.test-nesting-rules:not(#does-not-exist) + p,#test-is-pseudo + p {
 		order: 8;
 	}