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

feat(stylelint-config): allow zero units for css props #817

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

dannobytes
Copy link
Contributor

🚥 Resolves ISSUE_ID

🧰 Changes

Allows zero length units when defined by custom CSS properties. There are
cases where we often use the "calc" function to perform operations on CSS
properties, e.g. calc(100% - var(--sponge) - var(--bob)). Those always
require units to work properly, including 0 values.

https://stylelint.io/user-guide/rules/length-zero-no-unit

🧬 QA & Testing

Provide as much information as you can on how to test what you've done.

Allows zero length units when defined by custom CSS properties. There are
cases where we often use the "calc" function to perform operations on CSS
properties, e.g. `calc(100% - var(--sponge) - var(--bob))`. Those always
require units to work properly, including 0 values.

https://stylelint.io/user-guide/rules/length-zero-no-unit
@dannobytes dannobytes requested a review from erunion February 27, 2024 18:54
@erunion erunion merged commit 15ab8d7 into main Feb 27, 2024
2 checks passed
@erunion erunion deleted the daniels/allow-zero-units-css-props branch February 27, 2024 19:04
@erunion
Copy link
Member

erunion commented Feb 27, 2024

@dannobytes ive published this to @readme/[email protected]

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