Skip to content

Add mobile top nav tabs menu - #3888

Open
theletterf wants to merge 5 commits into
mainfrom
feature/mobile-top-nav-tabs
Open

Add mobile top nav tabs menu#3888
theletterf wants to merge 5 commits into
mainfrom
feature/mobile-top-nav-tabs

Conversation

@theletterf

Copy link
Copy Markdown
Member

Why

The section-driven top nav currently relies on a horizontal desktop tab row, which is cramped on narrow screens. A compact mobile presentation keeps configured sections reachable without changing the desktop layout.

What

This adds a mobile disclosure menu for top-nav link tabs below the desktop breakpoint, keeps the existing horizontal tab row for larger screens, and covers the rendering behavior with navigation tests.

The section-driven top nav needs a compact mobile presentation so configured tabs remain usable on narrow screens while preserving the existing desktop tab row.

Co-Authored-By: GPT-5.5 <noreply@openai.com>
The mobile top-nav menu should describe the documentation landing page when no section tab is active instead of using a generic sections label.

Co-Authored-By: GPT-5.5 <noreply@openai.com>
Keep the mobile section selector and version selector with the existing pages navigation drawer so small screens have a single navigation entry point.

Co-Authored-By: GPT-5.5 <noreply@openai.com>
Restore the mobile TOC trigger alignment now that version controls live inside the drawer.

Co-Authored-By: GPT-5.5 <noreply@openai.com>
Render the mobile-only pages drawer trigger on the docs landing page so section and version controls remain reachable there.

Co-Authored-By: GPT-5.5 <noreply@openai.com>
@theletterf

Copy link
Copy Markdown
Member Author

Latest update: the docs landing page now gets the same mobile pages drawer entry point as regular docs pages.

The landing layout renders a mobile-only hamburger trigger on the left and mounts the pages drawer only on mobile, so the section selector and version selector remain reachable without introducing the desktop sidebar on the landing page.

Verified locally with:

rtk dotnet test tests/Navigation.Tests/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant