Update sass to work around CRA build crashes #2643
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WHY are these changes introduced?
Fixes #2629
The current version of CRA' s build uses a version of https://github.com/postcss/postcss-custom-properties that crashes when it comes across
var()s inside calc()sa very specific order of calls:top:calc(var(--control-border-width)*-1)
. See #2629 (comment) for more info. This has been fixed in the newest version of postcss-custom-properties but that's not made its way into CRA (via postcss-preset-env) and probably won't be for some time.WHAT is this pull request doing?
Flip the order of values (yes I know this sounds silly)
How to 🎩
Ask @dleroux nicely to see that there are no visual regressions.
yarn run build
yarn create react-app testing
yarn run build