Skip to content

Conversation

jorytindall
Copy link
Contributor

@jorytindall jorytindall commented Jul 30, 2025

📌 Summary

Updates the default breakpoint of the AppHeader from lg (1088px) to md (768px).

🛠️ Detailed description

During the implementation of the Enterprise Navigation components, initial testing has solicited feedback around the default breakpoint at which the AppHeader collapses being too large; e.g., the elements in the AppHeader collapse into the small viewport menu at too large a size.

This presents some usability issues and can make it cumbersome to access the context switcher and other high-level navigation elements even at reasonable viewport widths (for example, when a desktop user has multiple tabs open next to one-another).

This was discussed in Slack here.

While potentially out-of-date (or anecdotal), we do have viewport usage metrics from TFC which validates that this change will result in the AppHeader being displayed in its uncollapsed state for >95% of users.

What this does not account for nor solve is very long strings in the context switcher that might result in wrapping. However, we believe this to be an edge case and the resulting experience is not broken.

The AppSideNav still collapses at the lg breakpoint, so while each component shifts at different breakpoints, it's logical to to keep the AppSideNav the same to free up space in the main page.

🔗 External links

Jira ticket: HDS-5164
Figma file: Enterprise Nav Breakpoints


👀 Component checklist

💬 Please consider using conventional comments when reviewing this PR.

📋 PCI review checklist
  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.
  • If applicable, I've worked with GRC to document the impact of any changes to security controls.
    Examples of changes to controls include access controls, encryption, logging, etc.
  • If applicable, I've worked with GRC to ensure compliance due to a significant change to the in-scope PCI environment.
    Examples include changes to operating systems, ports, protocols, services, cryptography-related components, PII processing code, etc.

@jorytindall jorytindall requested a review from a team July 30, 2025 20:42
@jorytindall jorytindall requested a review from a team as a code owner July 30, 2025 20:42
Copy link

vercel bot commented Jul 30, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
hds-showcase ✅ Ready (Inspect) Visit Preview Jul 30, 2025 10:53pm
hds-website ✅ Ready (Inspect) Visit Preview Jul 30, 2025 10:53pm

Copy link
Contributor

@heatherlarsen heatherlarsen left a comment

Choose a reason for hiding this comment

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

Nice!

Copy link
Contributor

@zamoore zamoore left a comment

Choose a reason for hiding this comment

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

Nice

@jorytindall jorytindall merged commit 0cc37b6 into main Jul 31, 2025
16 checks passed
@jorytindall jorytindall deleted the enterprise-nav-breakpoints branch July 31, 2025 23:08
@hashibot-hds hashibot-hds mentioned this pull request Jul 31, 2025
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.

6 participants