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

Docs improvement: Move search box up the sidebar #2072

Merged
merged 1 commit into from
Dec 5, 2022
Merged

Docs improvement: Move search box up the sidebar #2072

merged 1 commit into from
Dec 5, 2022

Conversation

fhightower
Copy link
Contributor

Currently on non-index pages, the search box is the last thing to appear in the sidebar. When viewing a page with a large "Table of Contents" section (example), the search box is very low on the page and readers must scroll down for a long time to find the search box. I recommend moving the search box up the sidebar so it appears 'above the fold' on most resolutions.

Let me know if you have any questions. Thanks!

Currently on non-index pages, the search box is the last thing to appear in the sidebar. When viewing a page with a large "Table of Contents" section ([example](https://marshmallow.readthedocs.io/en/stable/marshmallow.fields.html)), the search box is very low on the page and readers must scroll down for a long time to find the search box. I recommend moving the search box up the sidebar so it appears '[above the fold](https://en.wikipedia.org/wiki/Above_the_fold)' on most resolutions.

Let me know if you have any questions. Thanks!
@fhightower
Copy link
Contributor Author

Feel free to merge whenever you have a chance - I don't have permission to merge. Thanks!

@lafrech lafrech merged commit f8e97d8 into marshmallow-code:dev Dec 5, 2022
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.

None yet

2 participants