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
Add media-feature-range-notation
autofix
#6742
Add media-feature-range-notation
autofix
#6742
Conversation
🦋 Changeset detectedLatest commit: b854a03 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 |
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.
@romainmenke Thanks for the pull request. Great work! 👏🏼
I've left a few suggestions, but this change has no big problems. 👍🏼
Co-authored-by: Masafumi Koba <473530+ybiquitous@users.noreply.github.com>
Co-authored-by: Masafumi Koba <473530+ybiquitous@users.noreply.github.com>
Co-authored-by: Masafumi Koba <473530+ybiquitous@users.noreply.github.com>
Thank you for the review and feedback @ybiquitous 🙇 |
@@ -228,7 +228,7 @@ Enforce one representation of things that have multiple with these `notation` (s | |||
- [`hue-degree-notation`](../../lib/rules/hue-degree-notation/README.md): Specify number or angle notation for degree hues (Autofixable) (Ⓢ). | |||
- [`import-notation`](../../lib/rules/import-notation/README.md): Specify string or URL notation for `@import` rules (Autofixable) (Ⓢ). | |||
- [`keyframe-selector-notation`](../../lib/rules/keyframe-selector-notation/README.md): Specify keyword or percentage notation for keyframe selectors (Autofixable) (Ⓢ). | |||
- [`media-feature-range-notation`](../../lib/rules/media-feature-range-notation/README.md): Specify context or prefix notation for media feature ranges. | |||
- [`media-feature-range-notation`](../../lib/rules/media-feature-range-notation/README.md): Specify context or prefix notation for media feature ranges (Autofixable) (Ⓢ). |
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.
[note] Good catch to support the standard config. 👍🏼
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.
Wow, thanks for your quick fix! LGTM 👍🏼
Unable to parse when : less file: @media screen and(width <= @screen-sm-max) {
.topbox {
// code
}
} Need to add spaces manually: @media screen and (width <= @screen-sm-max) {
.topbox {
// code
}
} |
@yoyo837 Thanks for the report. Can you open a new issue with reproducible info, please? |
Closes #6501
Some bits are more complex/messy than I would like but I held of on trying to abstract this.
I personally prefer to only abstract away complexity after having found ±3 use cases.
But I don't mind spending time on refactors now if people with more experience in this code base have good ideas on how to improve this :)