-
-
Notifications
You must be signed in to change notification settings - Fork 8.4k
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
chore(v2): upgrade Infima #2669
Conversation
@@ -66,7 +66,7 @@ function NavItem({items, position, ...props}) { | |||
<ul className="dropdown__menu"> | |||
{items.map((linkItemInner, i) => ( | |||
<li key={i}> | |||
<NavLink className="navbar__item navbar__link" {...linkItemInner} /> |
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.
Need to change this otherwise the carets will also show up for the dropdown items. We lose the active highlighting here sadly.
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.
Uh-huh, so for dropdown items we need to use only one CSS class of dropdown__link
. Then we need to add an active dropdown item class to Infima and then specify it in the NavBar component. Right? I will fix it.
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.
The fix is ready, I'm waiting for the new version of Infima, which will define CSS class for active dropdown item.
Deploy preview for docusaurus-2 ready! Built with commit ca4a94a |
a538d14
to
85c6f4e
Compare
Motivation
Upgrade to
infima@0.2.0-alpha.8
which brings about some typography changes and navbar caret.Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
See Netlify.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)