Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix missing spaces around arithmetic operators #8615

Merged
merged 5 commits into from Jun 13, 2022
Merged

Fix missing spaces around arithmetic operators #8615

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
ad7a74f
add tests for spaced around operators in CSS math functions
dcastil Jun 13, 2022
81c6f12
fix CSS math functons besides calc not getting the love they deserve
dcastil Jun 13, 2022
d481036
improve comment
dcastil Jun 13, 2022
836b6db
update changelog
RobinMalfait Jun 13, 2022
ac4040e
update changelog
RobinMalfait Jun 13, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
3 changes: 3 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Expand Up @@ -7,9 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

### Fixed

- Fix extraction of multi-word utilities with arbitrary values and quotes ([#8604](https://github.com/tailwindlabs/tailwindcss/pull/8604))
- Fix casing of import of `corePluginList` type definition ([#8587](https://github.com/tailwindlabs/tailwindcss/pull/8587))
- Ignore PostCSS nodes returned by `addVariant` ([#8608](https://github.com/tailwindlabs/tailwindcss/pull/8608))
- Fix missing spaces around arithmetic operators ([#8615](https://github.com/tailwindlabs/tailwindcss/pull/8615))

## [3.1.2] - 2022-06-10

Expand Down
4 changes: 2 additions & 2 deletions src/util/dataTypes.js
View file
Open in desktop
Expand Up @@ -40,9 +40,9 @@ export function normalize(value, isRoot = true) {
value = value.trim()
}

// Add spaces around operators inside calc() that do not follow an operator
// Add spaces around operators inside math functions like calc() that do not follow an operator
// or '('.
value = value.replace(/calc\(.+\)/g, (match) => {
value = value.replace(/(calc|min|max|clamp)\(.+\)/g, (match) => {
return match.replace(
/(-?\d*\.?\d(?!\b-.+[,)](?![^+\-/*])\D)(?:%|[a-z]+)?|\))([+\-/*])/g,
'$1 $2 '
Expand Down
5 changes: 4 additions & 1 deletion tests/normalize-data-types.test.js
View file
Open in desktop
Expand Up @@ -20,7 +20,7 @@ let table = [
['var(--foo)', 'var(--foo)'],
['var(--headings-h1-size)', 'var(--headings-h1-size)'],

// calc(…) get's spaces around operators
// math functions like calc(…) get spaces around operators
['calc(1+2)', 'calc(1 + 2)'],
['calc(100%+1rem)', 'calc(100% + 1rem)'],
['calc(1+calc(100%-20px))', 'calc(1 + calc(100% - 20px))'],
Expand All @@ -29,6 +29,9 @@ let table = [
'calc(var(--headings-h1-size)*calc(100%+50%))',
'calc(var(--headings-h1-size) * calc(100% + 50%))',
],
['min(1+2)', 'min(1 + 2)'],
['max(1+2)', 'max(1 + 2)'],
['clamp(1+2,1+3,1+4)', 'clamp(1 + 2,1 + 3,1 + 4)'],
['var(--heading-h1-font-size)', 'var(--heading-h1-font-size)'],
['var(--my-var-with-more-than-3-words)', 'var(--my-var-with-more-than-3-words)'],
['var(--width, calc(100%+1rem))', 'var(--width, calc(100% + 1rem))'],
Expand Down