fix: convert value for opacity consistency

[anikethsaha]

fix #829

Summary

• If the opacity value is greater than 1, then fallback to 1
• if the opacity value is greater than 1 and it is a percentage, then keep it as it is

[codecov-io]

Codecov Report

Merging #842 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #842   +/-   ##
=======================================
  Coverage   97.65%   97.65%           
=======================================
  Files         111      111           
  Lines        3333     3333           
  Branches     1009     1010    +1     
=======================================
  Hits         3255     3255           
  Misses         71       71           
  Partials        7        7

Impacted Files	Coverage Δ
packages/postcss-convert-values/src/index.js	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d4af903...1619936. Read the comment docs.

[alexander-akait]

hm, it is invalid CSS syntax

[alexander-akait]

invalid syntax === invalid minification

[anikethsaha]

you meant the % is invalid ?

[alexander-akait]

Yes, try to use this in browser

[anikethsaha]

ohh,....I guess it should be then converted into decimal

opacity : 40

⬇️

opacity : 0.4

[alexander-akait]

@anikethsaha we are not linter or something like what handle incorrect syntax

[anikethsaha]

we are not linter or something like what handle incorrect syntax

😁 yea make sense

👍 closing

I guess good to close #829 too then !

[interactiveRob]

Actually percentage is valid syntax and it does work in the browser. Not sure if this has changed somewhat recently. I am running into the same issue on my project using a third-party library that uses % values for opacity.

https://developer.mozilla.org/en-US/docs/Web/CSS/opacity