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
Fix declaration-block-no-shorthand-property-overrides
false negatives for font
and border
#7606
Conversation
🦋 Changeset detectedLatest commit: 5bf2f29 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
lib/rules/declaration-block-no-shorthand-property-overrides/index.mjs
Outdated
Show resolved
Hide resolved
e7b3d80
to
23cbaeb
Compare
23cbaeb
to
3175dad
Compare
…es for `font` and `border`
3175dad
to
ccbc604
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Mouvedia Thanks for the pull request, and sorry for the late response. I have been very busy since last week.
I just left some suggestions. Can you take a look?
lib/rules/declaration-block-no-shorthand-property-overrides/index.mjs
Outdated
Show resolved
Hide resolved
1564f4c
to
e53937c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the quick fix. Can you address additional reviews?
lib/rules/declaration-block-no-shorthand-property-overrides/index.mjs
Outdated
Show resolved
Hide resolved
lib/rules/declaration-block-no-shorthand-property-overrides/index.mjs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. LGTM 👍🏼
…es for `font` and `border` (stylelint#7606)
Closes #7586
followup of #7585
border
resetsborder-image
which itself resets its own longhandssee https://www.w3.org/TR/css-backgrounds-3/#border-image
font-variant
addition is covered by #7620see https://github.com/stylelint/stylelint/pull/7606/files#diff-95fc42ab5cb27ff75b1ed1b46186e75f13e8fad2bfb54491d1c3fc4a478c0461R211