-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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(header): reset default ul styling directly in header navigation menus #4674
fix(header): reset default ul styling directly in header navigation menus #4674
Conversation
Deploy preview for carbon-elements ready! Built with commit f9d3d2a |
Deploy preview for the-carbon-components ready! Built with commit f9d3d2a https://deploy-preview-4674--the-carbon-components.netlify.com |
Deploy preview for carbon-components-react ready! Built with commit f9d3d2a https://deploy-preview-4674--carbon-components-react.netlify.com |
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.
Seems good to me!
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
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 @jendowns!
Closes #4673
This PR applies 3 missing style rules to the
HeaderNavigation
component'sul.bx--header__menu-bar
&ul.bx--header__menu
elements.Currently the component relies on global reset rules to render correctly, but for consumers who don't use those global resets (like us), the
ul.bx--header__menu-bar
&ul.bx--header__menu
elements still have padding, margins, and default list-style rules applied.Changelog
New
ul.bx--header__menu-bar
&ul.bx--header__menu