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.

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

Fix parsing of theme() inside calc() when there are no spaces around operators #11157

Merged
merged 12 commits into from May 4, 2023

Conversation

thecrypticace
Copy link
Contributor

This is ultimately a bug in postcss-value-parser so we've inlined the package and tweaked the parsing to handle this situation.

Fixes #11027

@thecrypticace thecrypticace merged commit 960cb40 into 3.3 May 4, 2023
21 checks passed
@thecrypticace thecrypticace deleted the fix/theme-fn-operators branch May 4, 2023 14:20
thecrypticace added a commit that referenced this pull request Jul 13, 2023
…und operators (#11157)

* Refactor

* Don’t resolve functions for anything not using theme or screen

* Normalize math operators inside calc when handling functions

* Inline postcss-value-parser

* Treat all functions the same as calc

* Remove workaround for calc + operators without spaces

* Remove `postcss-value-parser` dependency

* Update lockfile

* Update sourcemaps

* Update changelog

* Update `value-parser` formatting

* Stop prettier from complaining
thecrypticace added a commit that referenced this pull request Jul 13, 2023
…und operators (#11157)

* Refactor

* Don’t resolve functions for anything not using theme or screen

* Normalize math operators inside calc when handling functions

* Inline postcss-value-parser

* Treat all functions the same as calc

* Remove workaround for calc + operators without spaces

* Remove `postcss-value-parser` dependency

* Update lockfile

* Update sourcemaps

* Update changelog

* Update `value-parser` formatting

* Stop prettier from complaining
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant