Skip to content
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

Merged
merged 2 commits into from
Feb 7, 2018

Conversation

pixelastic
Copy link
Contributor

- Summary

The current live documentation website of Netlify CMS displays a scrollbar in the search dropdown menu:
image

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.
image

- 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)

image

@verythorough
Copy link
Contributor

verythorough commented Feb 5, 2018

Deploy preview for netlify-cms-www ready!

Built with commit 941a58f

https://deploy-preview-1080--netlify-cms-www.netlify.com

@verythorough
Copy link
Contributor

verythorough commented Feb 5, 2018

Deploy preview for cms-demo ready!

Built with commit 941a58f

https://deploy-preview-1080--cms-demo.netlify.com

@pixelastic
Copy link
Contributor Author

@verythorough Great talk at Fosdem yesterday by the way. You're really fast at deploying the previews! 😅

Copy link
Contributor

@tech4him1 tech4him1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tech4him1
Copy link
Contributor

tech4him1 commented Feb 5, 2018

@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.

@pixelastic
Copy link
Contributor Author

Done! ✅

@erquhart
Copy link
Contributor

erquhart commented Feb 7, 2018

@pixelastic p.s.: @verythorough isn't ignoring you, those deploy preview messages were generated by a bot she created :)

@erquhart erquhart merged commit 5b7fead into decaporg:master Feb 7, 2018
@pixelastic pixelastic deleted the website/header-link branch February 9, 2018 09:01
@verythorough
Copy link
Contributor

lol, @pixelastic Thanks!
(@erquhart I mentioned at my FOSDEM talk that people often think I'm just very fastidious about posting deploy preview comments. 😉 )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants