Skip to content

Add breadcrumb layout logic to fit header in one line#1521

Open
bghgary wants to merge 3 commits intoBabylonJS:masterfrom
bghgary:one-line-header
Open

Add breadcrumb layout logic to fit header in one line#1521
bghgary wants to merge 3 commits intoBabylonJS:masterfrom
bghgary:one-line-header

Conversation

@bghgary
Copy link
Contributor

@bghgary bghgary commented Feb 27, 2026

msedge_bS6bpScSTK

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request implements responsive breadcrumb navigation that automatically truncates breadcrumb items to fit the header in a single line. The implementation uses a hidden measuring container to calculate element widths and a ResizeObserver to dynamically adjust which breadcrumbs are visible based on available space.

Changes:

  • Added breadcrumb layout calculation logic using refs and ResizeObserver to dynamically show/hide breadcrumbs
  • Implemented a hidden measuring container to determine actual rendered widths of breadcrumb items
  • Added ellipsis indicator ("…") when breadcrumbs are truncated
  • Updated CSS to prevent wrapping and enable proper text overflow handling

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bjsplat
Copy link

bjsplat commented Feb 27, 2026

snapshot for this PR available at:

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/DOCS/refs/pull/1521/merge

@bghgary bghgary marked this pull request as ready for review February 27, 2026 17:51
@bjsplat
Copy link

bjsplat commented Feb 27, 2026

snapshot for this PR available at:

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/DOCS/refs/pull/1521/merge

@bghgary bghgary enabled auto-merge (squash) February 27, 2026 18:17
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