Update docs navigation to horizontal#3303
Open
m-hulbert wants to merge 6 commits into
Open
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
m-hulbert
force-pushed
the
docs-nav-update
branch
3 times, most recently
from
7fd8d23 to
b72d35b
Compare
m-hulbert
force-pushed
the
docs-nav-update
branch
from
7d72699 to
80436c3
Compare
m-hulbert
force-pushed
the
docs-nav-update
branch
from
80436c3 to
84d8ce1
Compare
m-hulbert
force-pushed
the
docs-nav-update
branch
from
84d8ce1 to
21ac1f5
Compare
m-hulbert
force-pushed
the
docs-nav-update
branch
from
21ac1f5 to
dd161f3
Compare
m-hulbert
changed the title
jamiehenson
requested changes
Apr 24, 2026
Member
jamiehenson
left a comment
jamiehenson
left a comment
There was a problem hiding this comment.
I like it, it's close! Sat with it a little more since demo earlier. Code is decent. I like the simplifications, I like the break up of the utility bar and the shuffling round of various things. Product bar works well. Pretty minor stuff from here tbh and some of it is subjective.
Small observations:
- I understand why the left nav contents was padded in (to align with the ably logo and the product row), but it looks weird and overly big to me, particularly when the padding between elements is so small. Honestly, I'd lower the side and top padding
- Product row on mobile looks a bit padded in as well, weird when scrolling
m-hulbert
added
review-app
m-hulbert
force-pushed
the
docs-nav-update
branch
from
dd161f3 to
cc6be6d
Compare
m-hulbert
added
review-app
jamiehenson
approved these changes
May 29, 2026
Member
jamiehenson
left a comment
jamiehenson
left a comment
There was a problem hiding this comment.
Good work! A nice evolution of the current UX
Replace the previous header navigation with three top-level tabs: Platform, Products, and Examples. Add a search button trigger with keyboard shortcut hint, Ask AI button, and help resources dropdown. Restructure the mobile menu with a tab-based layout where the Products tab keeps the ProductBar pinned above the scrollable sidebar. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add a sticky ProductBar for switching between products (Pub/Sub, Chat, Spaces, etc.) shown below the header on non-platform pages. Restructure the main layout from a centred container to a full-width flex layout with left sidebar, content pane (max 860px for docs, 1024px for examples), and right sidebar. Change button hover style from border colour to background fill. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Extract the copy-for-LLM functionality from PageHeader into a dedicated CopyForLLM component with a split button (copy action + dropdown for markdown preview, copy, and LLM links). Add ButtonGroup and Separator UI primitives. Simplify PageHeader after extraction. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Flatten the left sidebar to show a single product's nav with static section headings and accordion sub-menus. Move the language selector from the left sidebar to the right sidebar, restyle it as a bordered button matching the secondary button style. Add useShowLanguageSelector hook to determine visibility based on page languages. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ooks Extract the duplicated product bar visibility check into activePage.hasProductBar, computed once in the layout context. Simplify useShowLanguageSelector double negation to the equivalent some(). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Restructure nav data for all products to use section-based grouping with Getting started sections. Update generateMarkdownFooter test to match the new section structure. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
m-hulbert
force-pushed
the
docs-nav-update
branch
from
9e523bf to
294289c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR updates the docs navigation;