Skip to content
This repository has been archived by the owner on Feb 18, 2022. It is now read-only.

Upgrade from 6.2 to 7.0 breaks var() parsing #98

Closed
fivenp opened this issue Feb 20, 2018 · 3 comments
Closed

Upgrade from 6.2 to 7.0 breaks var() parsing #98

fivenp opened this issue Feb 20, 2018 · 3 comments

Comments

@fivenp
Copy link

fivenp commented Feb 20, 2018

Just upgraded from 6.2 to 7.0 and the parsing of color: color(var(--color-text)); breaks.

This used to be parsed into color: #000000 correctly with 6.2
With 7.0 the result actually is color: color(var(--color-text));

I guess the regex change over here might have caused this behaviour?
7cbf239

image

@jonathantneal
Copy link
Member

jonathantneal commented Feb 20, 2018

That’s not where that regex is used. It’s used for everything between var( and ). See https://github.com/postcss/postcss-custom-properties/blob/master/index.js#L50

Are you using cssnext? You might be having other issues.

@jonathantneal
Copy link
Member

@fivenp, any chance you are experiencing this issue?

#99 (comment)

@jonathantneal
Copy link
Member

Closing due to inactivity.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants