Skip to content

Conversation

@ar53n
Copy link
Contributor

@ar53n ar53n commented Mar 29, 2019

Hi everybody, i have same problem #49, but I am using postcss-color-function with postcss-css-variables
But unfortunately the browser does not handle as expected.
I think we can delete declarations with custom properties like a option.

expect:

.box {
  background-color: rgb(255, 128, 128); /* browser use that value */
  background-color: color(var(--red) tint(50%)); /* error value and browser skip it */
}

behavior:

.box {
  background-color: rgb(255, 128, 128); /* browser skip that value */
  background-color: color(var(--red) tint(50%)); /* work but not correct */
}

Arsen Karapetyan added 4 commits March 30, 2019 01:42
Add option preserveCustomProps.
When you set up preserveCustomProps to `false`
plugin delete all delcarations with custom properties for color()
Add package for more readable stdout
@jonathantneal jonathantneal merged commit b593011 into postcss:master Mar 30, 2019
@jonathantneal
Copy link
Member

@ar53n, while I don’t do regular maintainance on this plugin, this PR is clean, and I will merge it. Follow up with me if you would like a minor release.

@ar53n
Copy link
Contributor Author

ar53n commented Mar 31, 2019

@jonathantneal hello, can we to do minor or patch release for package?

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.

2 participants