Skip to content

Remove old nav code, make the new nav the default - #3381

Merged
rosslovas merged 1 commit into
mainfrom
ross/default-new-top-nav
Aug 19, 2026
Merged

Remove old nav code, make the new nav the default#3381
rosslovas merged 1 commit into
mainfrom
ross/default-new-top-nav

Conversation

@rosslovas

@rosslovas rosslovas commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Summary

We've been able to try out the new nav for a while with ?newnav appended to the URL, but the old header was still the experience customers saw, as the new nav was missing functionality.

I believe at this point the new nav has everything we need, so this PR attempts to delete all the code related to the old header, and makes the new header the default.

Results

Test it out at: https://stoctodocspr3381.z22.web.core.windows.net/docs

To test the signed-in state:

document.cookie = 'OctopusSignedInUser=' + encodeURIComponent(JSON.stringify({fullName: 'John Doe', email: 'jd@example.com'})) + ';path=/';
window.dispatchEvent(new CustomEvent('octopus:user-changed'));

(revert to the signed-out state with:)

document.cookie = 'OctopusSignedInUser=;path=/;max-age=0';
window.dispatchEvent(new CustomEvent('octopus:user-changed'));

Note that there are two relatively distinct experiences worth testing: the ordinary wide experience, and the mobile-friendly experience that appears when the viewport is less than 1130px wide.

@rosslovas
rosslovas force-pushed the ross/default-new-top-nav branch from 0f367a1 to 9317f3b Compare August 18, 2026 23:27
@team-marketing-branch-protections

Copy link
Copy Markdown

Pull request environment is available at https://stoctodocspr3381.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.

@rosslovas
rosslovas force-pushed the ross/default-new-top-nav branch from 9317f3b to d3bb78a Compare August 18, 2026 23:50
@rosslovas
rosslovas force-pushed the ross/default-new-top-nav branch from d3bb78a to 4972472 Compare August 19, 2026 00:21
@rosslovas
rosslovas marked this pull request as ready for review August 19, 2026 00:38
@rosslovas
rosslovas merged commit 1ea99c3 into main Aug 19, 2026
7 checks passed
@rosslovas
rosslovas deleted the ross/default-new-top-nav branch August 19, 2026 02:29
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.

3 participants