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

Use the full screen height for the main content to stabilize the nav #9153

Open
wants to merge 3 commits into
base: zb/docs-collapsible-project
Choose a base branch
from

Conversation

zanieb
Copy link
Member

@zanieb zanieb commented Nov 15, 2024

On large screens, we require scrolling below the fold for the next page / prev page navigation footer. This dramatically improves visibility of the left nav when looking at small pages like section overviews. Critically, this stops the height of the navigation from jumping around depending on the page you're on. On small screens, the positioning is unchanged since the nav is in a hamburger menu and it'd be annoying to scroll.

Eventually, we could move the next / prev nav out of the footer and into the content, e.g., as in #9121 (comment).

These images don't quite do the change in experience justice. It's the consistency when changing pages that feels the most different.

Before

Screenshot 2024-11-15 at 10 16 30 AM

After

Screenshot 2024-11-15 at 10 15 26 AM

@zanieb zanieb added the documentation Improvements or additions to documentation label Nov 15, 2024
@zanieb
Copy link
Member Author

zanieb commented Nov 15, 2024

cc @dhruvmanila

@zanieb zanieb force-pushed the zb/docs-collapsible-project branch 2 times, most recently from fd5f4f3 to c5ae128 Compare November 15, 2024 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant