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
docs: remove duplicate style properties #16226
Conversation
✅ Deploy Preview for docs-eslint ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
LGTM, thanks!
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.
Most of these are intentional duplications: one for browsers that support CSS variables and one for browsers that do not. I think we should leave these in place for best compatibility.
@@ -112,7 +111,6 @@ | |||
} | |||
|
|||
.search .search-results__item__title { | |||
font-size: var(--step-0); |
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.
This is intentionally duplicate. The first is used when the browser support variables and the second is used when the browser does not support variables.
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.
Added intentional properties and removed only duplicate properties & fixed typos.
font-size: .875rem; | ||
font-size: var(--step-0); |
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.
Can we keep the original order here, with the variable version coming first?
|
||
:root { |
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.
Was this removed accidentally?
@amareshsm are you going to get back to this PR? If not, we can close it. |
Closing in favor of #16466 |
Prerequisites checklist
What is the purpose of this pull request? (put an "X" next to an item)
[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofix to a rule
[ ] Add a CLI option
[ ] Add something to the core
[x] Other, please explain:
In the documentation website removed duplicate style properties.
What changes did you make? (Give an overview)
In the documentation website removed duplicate style properties.