Skip to content

Re-style the right-hand article navigation panel - #3318

Open
borland wants to merge 4 commits into
mainfrom
orion/right-nav
Open

Re-style the right-hand article navigation panel#3318
borland wants to merge 4 commits into
mainfrom
orion/right-nav

Conversation

@borland

@borland borland commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Results

Claude does all the coding here; I reviewed and re-steered it several times to improve things.

  • Updates the styling of the right hand article nav panel to match the figma design
  • Overhauls the javascript which hilights the selected area as you scroll. The existing code had several bugs and would sometimes hilight a section other than the one you had clicked on. This bug exists today on octopuscom/docs but it became easier to trigger with the slightly smaller font sizes in the style update.
    • It now updates instantly as you scroll, rather than on a 1000ms timer
    • It now reflects the position within the document accurately

Before

image

After

image

Testing

Tested on macOS in MS Edge (chrome), Safari and Firefox

import { accelerator } from '@lib/accelerator';
import { SITE } from '@config';
import { Translations, Lang } from '@util/Languages';
import Separator from './Separator.astro';

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can this be deleted now

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah looks like it's unused except for the components demo page.

I think I'd prefer to leave it though, in case we need to put it back for some reason, and file a card to clean it up at the end, if that's OK?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Actually, given that the spelling error caused me to need to re-spin the PR, I deleted the component.

Claude keeps writing comments in the CSS which say "colour" but our spelling requires AMERICAN :-(

@team-marketing-branch-protections

Copy link
Copy Markdown

Pull request environment is available at https://stoctodocspr3318.z22.web.core.windows.net.

You can view the ephemeral environment status in Octopus Deploy.

This environment will be automatically deprovisioned when the pull request is closed, or after 7 days of inactivity.

@enf0rc3

enf0rc3 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Looks all good overall, would be happy with merge, one thing i did notice is we have lost toe header highlighted when its expanded and the hover affect like the original, also the carat is facing right when its collapsed instead of down saying, click to pull the menu down, all trivial.

image image

can see here, side by side view (zoom in on both to trigger the responsive view.)
Before: https://octopus.com/docs/installation
After: https://stoctodocspr3318.z22.web.core.windows.net/docs/installation

Oh and there is also a c-spell error

@borland

borland commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

one thing i did notice is we have lost toe header highlighted when its expanded and the hover affect like the original

I'm not sure what this means?

also the carat is facing right when its collapsed instead of down saying, click to pull the menu down, all trivial.

I didn't realize that the article nav event did anything in the mobile layout! Upon seeing it though, the change is intentional; In the figma it states for the nav that unexpanded sections should have a rightward-facing arrow. Down means expanded. The previous behaviour was wrong. Good job claude

@ellen-octopus

ellen-octopus commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Very nice @borland 👏🏻

I have two hopefully small bits of feedback:

  1. Can we make the gap between the body content and the article navigation 80px?
CleanShot 2026-08-07 at 15 24 01@2x
  1. This is a nice-to-have, so happy to put this in the backlog to address as polish, but would it be possible to make the active nav transition smooth - like the way Linear does it?
CleanShot.2026-08-07.at.15.26.18.mp4

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