Skip to content

fix(#404): With a selection, earlier breadcrumbs paint over the separator and the folder ti - #413

Merged
BeSovereign merged 1 commit into
mainfrom
fix/issue-404
Oct 20, 2026
Merged

BeSovereign merged 1 commit into
mainfrom
fix/issue-404

Conversation

@BeSovereign

Copy link
Copy Markdown
Contributor

Refs #404

While a selection lasts, every breadcrumb link is now cut with its own ellipsis (overflow hidden, text-overflow ellipsis, nowrap); earlier crumbs shrink first (flex-shrink 100), separators never shrink, and the current crumb keeps min-width 8rem, so no crumb's text runs into the '/' or the title. A new Playwright test in de-du at 1280x800 asserts that each crumb fits or is ellipsised, that the current crumb keeps 8rem and that crumb and separator boxes don't overlap; DESIGN.md and DECISIONS.md record the shrink order.

Driver gates (measured, not claimed)

  • Suites: typecheck: 1 passed, 0 failed | lint: 1 passed, 0 failed | unit: 1187 passed, 0 failed | e2e: 606 passed, 0 failed
  • Red/green check on declared tests: e2e/selection-bar.spec.ts
red/green output
RED without the change:
─────────────────────────────────────────────────────

    Error Context: test-results/selection-bar--404-with-a--da6b8--ellipses-and-never-overlap-desktop-chrome/error-context.md

    attachment #3: trace (application/zip) ─────────────────────────────────────────────────────────
    test-results/selection-bar--404-with-a--da6b8--ellipses-and-never-overlap-desktop-chrome/trace.zip
    Usage:

        pnpm exec playwright show-trace test-results/selection-bar--404-with-a--da6b8--ellipses-and-never-overlap-desktop-chrome/trace.zip

    ────────────────────────────────────────────────────────────────────────────────────────────────

  1 failed
    [desktop-chrome] › e2e/selection-bar.spec.ts:230:1 › #404 with a selection the German crumbs are cut with ellipses and never overlap 
  1 skipped


GREEN with the change:
p cannot interpret due to the position of the comment. The comment will be removed to avoid issues.

(!) Some chunks are larger than 500 kB after minification. Consider:
- Using dynamic import() to code-split the application
- Use build.rollupOptions.output.manualChunks to improve chunking: https://rollupjs.org/configuration-options/#output-manualchunks
- Adjust chunk size limit for this warning via build.chunkSizeWarningLimit.
$ node build.mjs


Running 2 tests using 1 worker

  ✓  1 [desktop-chrome] › e2e/selection-bar.spec.ts:230:1 › #404 with a selection the German crumbs are cut with ellipses and never overlap (1.1s)
  -  2 [mobile-chrome] › e2e/selection-bar.spec.ts:230:1 › #404 with a selection the German crumbs are cut with ellipses and never overlap

  1 skipped
  1 passed (4.9s)

🤖 Generated with Claude Code

While a selection lasts, every breadcrumb link is now cut with its own ellipsis (overflow hidden, text-overflow ellipsis, nowrap); earlier crumbs shrink first (flex-shrink 100), separators never shrink, and the current crumb keeps min-width 8rem, so no crumb's text runs into the '/' or the title. A new Playwright test in de-du at 1280x800 asserts that each crumb fits or is ellipsised, that the current crumb keeps 8rem and that crumb and separator boxes don't overlap; DESIGN.md and DECISIONS.md record the shrink order.

Refs #404

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@BeSovereign
BeSovereign merged commit b34651e into main Oct 20, 2026
3 checks passed
@BeSovereign
BeSovereign deleted the fix/issue-404 branch October 20, 2026 13:18
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.

1 participant