Skip to content

fix: restore bucket breadcrumb click navigation#135

Merged
cxymds merged 1 commit into
mainfrom
codex/fix-breadcrumb-click-browser-list
Jun 15, 2026
Merged

fix: restore bucket breadcrumb click navigation#135
cxymds merged 1 commit into
mainfrom
codex/fix-breadcrumb-click-browser-list

Conversation

@cxymds

@cxymds cxymds commented Jun 15, 2026

Copy link
Copy Markdown
Member

Summary

  • Render breadcrumb links directly through BreadcrumbLink instead of nesting Next Link via Base UI render.
  • Keep breadcrumb clicks routed through router.push so bucket detail pages can return to the bucket list after refresh.
  • Add a source regression test to prevent reintroducing the broken composition pattern.

Fixes rustfs/rustfs#3403

Test Plan

  • node --test tests/lib/top-nav-breadcrumb-source.test.js
  • COREPACK_ENABLE_PROJECT_SPEC=0 pnpm lint
  • COREPACK_ENABLE_PROJECT_SPEC=0 pnpm tsc --noEmit

@cxymds cxymds merged commit 8bd6ddc into main Jun 15, 2026
8 checks passed
@cxymds cxymds deleted the codex/fix-breadcrumb-click-browser-list branch June 15, 2026 03:13
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.

After entering the bucket, refreshing the page prevents you from returning to the bucket list.

1 participant