-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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 DocSearch link alignment #1080
Conversation
Deploy preview for netlify-cms-www ready! Built with commit 941a58f |
Deploy preview for cms-demo ready! Built with commit 941a58f |
@verythorough Great talk at Fosdem yesterday by the way. You're really fast at deploying the previews! 😅 |
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
@pixelastic Can you add yourself to the contributors list: https://github.com/netlify/netlify-cms/blob/master/CONTRIBUTING.md#add-yourself-as-a-contributor You can just push it to this PR if you want. |
Done! ✅ |
@pixelastic p.s.: @verythorough isn't ignoring you, those deploy preview messages were generated by a bot she created :) |
lol, @pixelastic Thanks! |
- Summary
The current live documentation website of Netlify CMS displays a scrollbar in the search dropdown menu:
This is due to the "Search by Algolia" link being slightly too offset on the right. This PR solves this by making the offset only apply to top-level links in the header, and not all links in the dropdown.
- Test plan
This is the new output after the slight change of markup. The content of the dropdown is correctly styled, while the top-level links keep their initial styling.
- Description for the changelog
Only applied link styling in the header to the top-level links.
- A picture of a cute animal (not mandatory but encouraged)