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

Change alpha-value-notation for opacity property #210

Closed
GrimLink opened this issue Nov 8, 2021 · 3 comments · Fixed by #212
Closed

Change alpha-value-notation for opacity property #210

GrimLink opened this issue Nov 8, 2021 · 3 comments · Fixed by #212
Labels
status: ask to implement ask before implementing as may no longer be relevant

Comments

@GrimLink
Copy link

GrimLink commented Nov 8, 2021

What steps are needed to reproduce the bug?

Use stylelint version 14 with stylelint-config-standard version 23.

What did you expect to happen?

No error on opacity: 0.8;

What actually happened?

It throws an error on opacities with a number value, while this not supported in every browser yet -> https://developer.mozilla.org/en-US/docs/Web/CSS/opacity

_tests_/index.scss
 59:14  ✖  Expected "0.8" to be "80%"  alpha-value-notation
@GrimLink
Copy link
Author

GrimLink commented Nov 8, 2021

This rule should not be part of this config or should be exclude the css property opacity

@jeddy3 jeddy3 changed the title alpha-value-notation throws error for css properties with alpha numbers Fix alpha-value-notation for opacity property Nov 9, 2021
@jeddy3 jeddy3 added status: ask to implement ask before implementing as may no longer be relevant type: bug labels Nov 9, 2021
@jeddy3
Copy link
Member

jeddy3 commented Nov 9, 2021

@GrimLink Thanks for the report.

Let's exclude the opacity property until it lands in the last two version of Safari.

I've labelled the issue as ready to implement. Please consider contributing if you have time.

@GrimLink
Copy link
Author

GrimLink commented Nov 9, 2021

I have time, and the PR is created 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: ask to implement ask before implementing as may no longer be relevant
2 participants