tailwindcss 3.2.5 → 3.2.7

package/CHANGELOG.md

@@ -9,6 +9,19 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - Nothing yet!
 
+## [3.2.7] - 2023-02-16
+
+### Fixed
+
+- Fix use of `:where(.btn)` when matching `!btn` ([#10601](https://github.com/tailwindlabs/tailwindcss/pull/10601))
+- Revert including `outline-color` in `transition` and `transition-colors` by default ([#10604](https://github.com/tailwindlabs/tailwindcss/pull/10604))
+
+## [3.2.6] - 2023-02-08
+
+### Fixed
+
+- drop oxide api shim ([add16364b4b1100e1af23ad1ca6900a0b53cbba0](https://github.com/tailwindlabs/tailwindcss/commit/add16364b4b1100e1af23ad1ca6900a0b53cbba0))
+
 ## [3.2.5] - 2023-02-08
 
 ### Added
@@ -2163,7 +2176,9 @@ No release notes
 
 - Everything!
 
-[unreleased]: https://github.com/tailwindlabs/tailwindcss/compare/v3.2.5...HEAD
+[unreleased]: https://github.com/tailwindlabs/tailwindcss/compare/v3.2.7...HEAD
+[3.2.7]: https://github.com/tailwindlabs/tailwindcss/compare/v3.2.6...v3.2.7
+[3.2.6]: https://github.com/tailwindlabs/tailwindcss/compare/v3.2.5...v3.2.6
 [3.2.5]: https://github.com/tailwindlabs/tailwindcss/compare/v3.2.4...v3.2.5
 [3.2.4]: https://github.com/tailwindlabs/tailwindcss/compare/v3.2.3...v3.2.4
 [3.2.3]: https://github.com/tailwindlabs/tailwindcss/compare/v3.2.2...v3.2.3

package/lib/lib/expandTailwindAtRules.js

@@ -13,7 +13,6 @@ const _generateRules = require("./generateRules");
 const _log = /*#__PURE__*/ _interopRequireDefault(require("../util/log"));
 const _cloneNodes = /*#__PURE__*/ _interopRequireDefault(require("../util/cloneNodes"));
 const _defaultExtractor = require("./defaultExtractor");
-const _oxide = /*#__PURE__*/ _interopRequireDefault(require("@tailwindcss/oxide"));
 function _interopRequireDefault(obj) {
     return obj && obj.__esModule ? obj : {
         default: obj
@@ -155,7 +154,7 @@ function expandTailwindAtRules(context) {
         env.DEBUG && console.time("Reading changed files");
         if (env.OXIDE) {
             // TODO: Pass through or implement `extractor`
-            for (let candidate of _oxide.default.parseCandidateStringsFromFiles(context.changedContent)){
+            for (let candidate of require("@tailwindcss/oxide").parseCandidateStringsFromFiles(context.changedContent)){
                 candidates.add(candidate);
             }
         // for (let { file, content, extension } of context.changedContent) {

package/lib/lib/generateRules.js

@@ -162,12 +162,12 @@ function applyImportant(matches, classCandidate) {
             ]
         });
         container.walkRules((r)=>{
-            r.selector = (0, _pluginUtils.updateAllClasses)((0, _pluginUtils.filterSelectorsForClass)(r.selector, classCandidate), (className)=>{
-                if (className === classCandidate) {
-                    return `!${className}`;
-                }
-                return className;
-            });
+            let ast = (0, _postcssSelectorParser.default)().astSync(r.selector);
+            // Remove extraneous selectors that do not include the base candidate
+            ast.each((sel)=>(0, _formatVariantSelector.eliminateIrrelevantSelectors)(sel, classCandidate));
+            // Update all instances of the base candidate to include the important marker
+            (0, _pluginUtils.updateAllClasses)(ast, (className)=>className === classCandidate ? `!${className}` : className);
+            r.selector = ast.toString();
             r.walkDecls((d)=>d.important = true);
         });
         result.push([

package/lib/util/formatVariantSelector.js

@@ -10,6 +10,7 @@ function _export(target, all) {
 }
 _export(exports, {
     formatVariantSelector: ()=>formatVariantSelector,
+    eliminateIrrelevantSelectors: ()=>eliminateIrrelevantSelectors,
     finalizeSelector: ()=>finalizeSelector,
     handleMergePseudo: ()=>handleMergePseudo
 });
@@ -98,18 +99,7 @@ function formatVariantSelector(formats, { context , candidate  }) {
     });
     return sel;
 }
-/**
- * Remove extraneous selectors that do not include the base class/candidate
- *
- * Example:
- * Given the utility `.a, .b { color: red}`
- * Given the candidate `sm:b`
- *
- * The final selector should be `.sm\:b` and not `.a, .sm\:b`
- *
- * @param {Selector} ast
- * @param {string} base
- */ function eliminateIrrelevantSelectors(sel, base) {
+function eliminateIrrelevantSelectors(sel, base) {
     let hasClassesMatchingCandidate = false;
     sel.walk((child)=>{
         if (child.type === "class" && child.value === base) {

package/lib/util/pluginUtils.js

@@ -10,7 +10,6 @@ function _export(target, all) {
 }
 _export(exports, {
     updateAllClasses: ()=>updateAllClasses,
-    filterSelectorsForClass: ()=>filterSelectorsForClass,
     asValue: ()=>asValue,
     parseColorFormat: ()=>parseColorFormat,
     asColor: ()=>asColor,
@@ -19,7 +18,6 @@ _export(exports, {
     coerceValue: ()=>coerceValue,
     getMatchingTypes: ()=>getMatchingTypes
 });
-const _postcssSelectorParser = /*#__PURE__*/ _interopRequireDefault(require("postcss-selector-parser"));
 const _escapeCommas = /*#__PURE__*/ _interopRequireDefault(require("./escapeCommas"));
 const _withAlphaVariable = require("./withAlphaVariable");
 const _dataTypes = require("./dataTypes");
@@ -32,29 +30,12 @@ function _interopRequireDefault(obj) {
     };
 }
 function updateAllClasses(selectors, updateClass) {
-    let parser = (0, _postcssSelectorParser.default)((selectors)=>{
-        selectors.walkClasses((sel)=>{
-            let updatedClass = updateClass(sel.value);
-            sel.value = updatedClass;
-            if (sel.raws && sel.raws.value) {
-                sel.raws.value = (0, _escapeCommas.default)(sel.raws.value);
-            }
-        });
-    });
-    let result = parser.processSync(selectors);
-    return result;
-}
-function filterSelectorsForClass(selectors, classCandidate) {
-    let parser = (0, _postcssSelectorParser.default)((selectors)=>{
-        selectors.each((sel)=>{
-            const containsClass = sel.nodes.some((node)=>node.type === "class" && node.value === classCandidate);
-            if (!containsClass) {
-                sel.remove();
-            }
-        });
+    selectors.walkClasses((sel)=>{
+        sel.value = updateClass(sel.value);
+        if (sel.raws && sel.raws.value) {
+            sel.raws.value = (0, _escapeCommas.default)(sel.raws.value);
+        }
     });
-    let result = parser.processSync(selectors);
-    return result;
 }
 function resolveArbitraryValue(modifier, validate) {
     if (!isArbitraryValue(modifier)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tailwindcss",
-  "version": "3.2.5",
+  "version": "3.2.7",
   "description": "A utility-first CSS framework for rapidly building custom user interfaces.",
   "license": "MIT",
   "main": "lib/index.js",
@@ -41,7 +41,6 @@
     "stubs/*.stub.js",
     "nesting/*",
     "types/**/*",
-    "oxide-node-api-shim",
     "*.d.ts",
     "*.css",
     "*.js"
@@ -71,7 +70,6 @@
     "postcss": "^8.0.9"
   },
   "dependencies": {
-    "@tailwindcss/oxide": "file:./oxide-node-api-shim",
     "arg": "^5.0.2",
     "chokidar": "^3.5.3",
     "color-name": "^1.1.4",

package/src/lib/expandTailwindAtRules.js

@@ -6,8 +6,6 @@ import log from '../util/log'
 import cloneNodes from '../util/cloneNodes'
 import { defaultExtractor } from './defaultExtractor'
 
-import oxide from '@tailwindcss/oxide'
-
 let env = sharedState.env
 
 const builtInExtractors = {
@@ -134,7 +132,7 @@ export default function expandTailwindAtRules(context) {
 
     if (env.OXIDE) {
       // TODO: Pass through or implement `extractor`
-      for (let candidate of oxide.parseCandidateStringsFromFiles(
+      for (let candidate of require('@tailwindcss/oxide').parseCandidateStringsFromFiles(
         context.changedContent
         // Object.assign({}, builtInTransformers, context.tailwindConfig.content.transform)
       )) {

package/src/lib/generateRules.js

@@ -3,10 +3,14 @@ import selectorParser from 'postcss-selector-parser'
 import parseObjectStyles from '../util/parseObjectStyles'
 import isPlainObject from '../util/isPlainObject'
 import prefixSelector from '../util/prefixSelector'
-import { updateAllClasses, filterSelectorsForClass, getMatchingTypes } from '../util/pluginUtils'
+import { updateAllClasses, getMatchingTypes } from '../util/pluginUtils'
 import log from '../util/log'
 import * as sharedState from './sharedState'
-import { formatVariantSelector, finalizeSelector } from '../util/formatVariantSelector'
+import {
+  formatVariantSelector,
+  finalizeSelector,
+  eliminateIrrelevantSelectors,
+} from '../util/formatVariantSelector'
 import { asClass } from '../util/nameClass'
 import { normalize } from '../util/dataTypes'
 import { isValidVariantFormatString, parseVariant } from './setupContextUtils'
@@ -111,22 +115,28 @@ function applyImportant(matches, classCandidate) {
   if (matches.length === 0) {
     return matches
   }
+
   let result = []
 
   for (let [meta, rule] of matches) {
     let container = postcss.root({ nodes: [rule.clone()] })
+
     container.walkRules((r) => {
-      r.selector = updateAllClasses(
-        filterSelectorsForClass(r.selector, classCandidate),
-        (className) => {
-          if (className === classCandidate) {
-            return `!${className}`
-          }
-          return className
-        }
+      let ast = selectorParser().astSync(r.selector)
+
+      // Remove extraneous selectors that do not include the base candidate
+      ast.each((sel) => eliminateIrrelevantSelectors(sel, classCandidate))
+
+      // Update all instances of the base candidate to include the important marker
+      updateAllClasses(ast, (className) =>
+        className === classCandidate ? `!${className}` : className
       )
+
+      r.selector = ast.toString()
+
       r.walkDecls((d) => (d.important = true))
     })
+
     result.push([{ ...meta, important: true }, container.nodes[0]])
   }
 

package/src/util/formatVariantSelector.js

@@ -120,7 +120,7 @@ function resortSelector(sel) {
  * @param {Selector} ast
  * @param {string} base
  */
-function eliminateIrrelevantSelectors(sel, base) {
+export function eliminateIrrelevantSelectors(sel, base) {
   let hasClassesMatchingCandidate = false
 
   sel.walk((child) => {

package/src/util/pluginUtils.js

@@ -1,4 +1,3 @@
-import selectorParser from 'postcss-selector-parser'
 import escapeCommas from './escapeCommas'
 import { withAlphaValue } from './withAlphaVariable'
 import {
@@ -21,37 +20,19 @@ import negateValue from './negateValue'
 import { backgroundSize } from './validateFormalSyntax'
 import { flagEnabled } from '../featureFlags.js'
 
+/**
+ * @param {import('postcss-selector-parser').Container} selectors
+ * @param {(className: string) => string} updateClass
+ * @returns {string}
+ */
 export function updateAllClasses(selectors, updateClass) {
-  let parser = selectorParser((selectors) => {
-    selectors.walkClasses((sel) => {
-      let updatedClass = updateClass(sel.value)
-      sel.value = updatedClass
-      if (sel.raws && sel.raws.value) {
-        sel.raws.value = escapeCommas(sel.raws.value)
-      }
-    })
-  })
-
-  let result = parser.processSync(selectors)
+  selectors.walkClasses((sel) => {
+    sel.value = updateClass(sel.value)
 
-  return result
-}
-
-export function filterSelectorsForClass(selectors, classCandidate) {
-  let parser = selectorParser((selectors) => {
-    selectors.each((sel) => {
-      const containsClass = sel.nodes.some(
-        (node) => node.type === 'class' && node.value === classCandidate
-      )
-      if (!containsClass) {
-        sel.remove()
-      }
-    })
+    if (sel.raws && sel.raws.value) {
+      sel.raws.value = escapeCommas(sel.raws.value)
+    }
   })
-
-  let result = parser.processSync(selectors)
-
-  return result
 }
 
 function resolveArbitraryValue(modifier, validate) {

package/stubs/defaultConfig.stub.js

@@ -877,8 +877,8 @@ module.exports = {
       none: 'none',
       all: 'all',
       DEFAULT:
-        'color, background-color, border-color, outline-color, text-decoration-color, fill, stroke, opacity, box-shadow, transform, filter, backdrop-filter',
-      colors: 'color, background-color, border-color, outline-color, text-decoration-color, fill, stroke',
+        'color, background-color, border-color, text-decoration-color, fill, stroke, opacity, box-shadow, transform, filter, backdrop-filter',
+      colors: 'color, background-color, border-color, text-decoration-color, fill, stroke',
       opacity: 'opacity',
       shadow: 'box-shadow',
       transform: 'transform',

package/oxide-node-api-shim/index.js

@@ -1,21 +0,0 @@
-let log = require('../lib/util/log').default
-
-// This should be a temporary file.
-//
-// Right now we require `@tailwindcss/oxide` as one of the packages in package.json. This contains
-// all the necessary Rust bindings. However, we won't ship those bindings by default yet, and
-// therefore you need to install the explicit oxide-insiders version where the Rust bindings are
-// available.
-//
-// To ensure that this doesn't break existing builds of the insiders release, we will use this shim
-// to implement all the APIs and show a warning in case you are trying to run `OXIDE=1 npx
-// tailwindcs ...` without having installed the oxide-insiders version.
-module.exports.parseCandidateStringsFromFiles = function parseCandidateStringsFromFiles(
-  _changedContent
-) {
-  log.warn('oxide-required', [
-    'It looks like you are trying to run Tailwind CSS with the OXIDE=1 environment variable.',
-    'This version does not have the necessary Rust bindings, so please install the `tailwindcss@insiders-oxide` version instead.',
-  ])
-  return []
-}

package/oxide-node-api-shim/package.json

@@ -1,5 +0,0 @@
-{
-  "name": "@tailwindcss/oxide-shim",
-  "main": "./index.js",
-  "license": "MIT"
-}