-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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 input group action static color #709
Conversation
color: $pt-text-color-muted; | ||
|
||
// same goes for dark | ||
// stylelint-disable selector-max-compound-selectors |
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.
stylelint-disable-next-line
automatically re-enables
|
||
// same goes for dark | ||
// stylelint-disable selector-max-compound-selectors | ||
.pt-dark & { color: $pt-dark-text-color-muted; } |
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.
nit: I prefer to format even single-rule blocks the normal way:
.pt-dark & {
color: $pt-dark-text-color-muted;
}
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.
i personally love a good one-liner, but the multiline syntax is definitely preferred in this codebase.
Before (lighter than placeholder, bad):
After: