Skip to content

Bolt 335 sub nav improvements#1209

Open
markdoran-ft wants to merge 10 commits intomainfrom
BOLT-335-sub-nav-improvements
Open

Bolt 335 sub nav improvements#1209
markdoran-ft wants to merge 10 commits intomainfrom
BOLT-335-sub-nav-improvements

Conversation

@markdoran-ft
Copy link
Copy Markdown
Contributor

@markdoran-ft markdoran-ft commented Mar 30, 2026

Useful information

Ticket: BOLT-335
Designs: Figma

Description

This PR introduces the following changes:

  1. Styles and behaviour of the sub navigation dropdown are now controlled by screen size and not whether the device supports hover or not.
  2. The dropdown can now contain a selected item that will display in teal.
  3. Various updates to the styling of the dropdowns including:
  • Reducing the spacing between list items
  • On mobile devices increasing the space between the edge of the window and the dropdown modal
  • Adding a transition before the dropdown is displayed. This transition will match the transition that is used when the meganav moves into view.
  • On smaller screens, where the dropdown with the close button is displayed, a scroll lock has now been added. You can scroll on the menu but cannot scroll the content behind the menu.
  1. Now uses o-header v 15.5.0 which updates the keyboard handling of the subnav dropdown on both mobile and desktop.

Screenshots

Updated desktop

Before After
Screenshot 2026-03-30 at 14 19 41 Screenshot 2026-03-30 at 14 08 58

Updated mobile

Before After
Screenshot 2026-03-30 at 14 19 11 Screenshot 2026-03-30 at 13 35 08

@markdoran-ft markdoran-ft marked this pull request as ready for review April 8, 2026 13:00
Copy link
Copy Markdown
Contributor

@Stegosauruss Stegosauruss left a comment

Choose a reason for hiding this comment

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

LGTM on mobile and desktop. Animation is consistent with meganav. Nice work

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.

2 participants