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-redundant-longhand-properties
false negatives for *-block
and *-inline
logical properties
#7208
Fix declaration-block-no-redundant-longhand-properties
false negatives for *-block
and *-inline
logical properties
#7208
Conversation
…ves for `margin-block` and `margin-inline` This is the original ask in #7200.
🦋 Changeset detectedLatest commit: 9b09883 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 |
…ves for `padding-block` and `padding-inline`
declaration-block-no-redundant-longhand-properties
false negativesdeclaration-block-no-redundant-longhand-properties
false negatives for *-block
and *-inline
logical properties
Going to mark this as ready-for-review since I think it's big enough! |
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.
Great. Thanks for making this change!
One question, should we update the property list in the rule's README?
This rule complains when the following shorthand properties can be used: |
00e2315
to
9b09883
Compare
Ah! Totally slipped my mind, thanks. Just added those and sorted the list alphabetically (I'm not sure what the previous order was). (accidentally force-pushed - sorry about that! only the latest commit is different from the last review) |
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.
Great. LGTM 👍🏼
Closes #7200.
Bundling many properties into this PR as suggested in the original issue. I'm only including the ones that are trailed with
-block
and-inline
(as scoped in the original issue title); I will address further false negatives in another PR.I've chosen to base this off of
main
since:v16
Affected properties:
border-block
border-inline
inset-block
inset-inline
scroll-margin-block
scroll-margin-inline
scroll-padding-block
scroll-padding-inline
margin-block
margin-inline
padding-block
padding-inline