Skip to content

fix(desktop): show GitHub vaults as ⎇ owner/repo@ref in the title breadcrumb#161

Merged
oratis merged 1 commit into
mainfrom
ux/desktop-github-vault-badge
Jun 14, 2026
Merged

fix(desktop): show GitHub vaults as ⎇ owner/repo@ref in the title breadcrumb#161
oratis merged 1 commit into
mainfrom
ux/desktop-github-vault-badge

Conversation

@oratis

@oratis oratis commented Jun 14, 2026

Copy link
Copy Markdown
Owner

Item 3/4 (desktop half) — the counterpart of #160.

Problem

The title-bar breadcrumb showed only the vault's folder basename, so a GitHub vault was indistinguishable from a local folder and the branch wasn't visible.

Fix

When the open vault is a materialized GitHub repo, the breadcrumb renders a branch marker + owner/repo@ref (with a "GitHub vault · …" tooltip) instead of the bare basename. Local folders unchanged. githubVault is already tracked in App.tsx; it's just passed into Toolbar.

Verify

tsc ✓ · vitest 9 (+1) ✓ · biome ✓ · build ✓.

Last item (dedupe re-open + clear-cache) follows.

🤖 Generated with Claude Code

@oratis oratis force-pushed the ux/desktop-github-vault-badge branch from 6fc01a2 to 88e8000 Compare June 14, 2026 05:59
…adcrumb

The title-bar breadcrumb showed only the vault's folder basename, so a GitHub
vault was indistinguishable from a local folder and the branch wasn't visible.

When the open vault is a materialized GitHub repo, the breadcrumb now renders a
branch marker + `owner/repo@ref` (with a "GitHub vault · …" tooltip) instead of
the bare basename. Local folders are unchanged. `githubVault` is already tracked
in App.tsx; it's just passed to the Toolbar.

tsc ✓ · vitest (9, +1) ✓ · biome ✓ · build ✓.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@oratis oratis force-pushed the ux/desktop-github-vault-badge branch from 88e8000 to 3d18f76 Compare June 14, 2026 06:04
@oratis oratis merged commit ceb93f2 into main Jun 14, 2026
4 checks passed
@oratis oratis deleted the ux/desktop-github-vault-badge branch June 14, 2026 06:09
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