Skip to content

feat: make switching repositories discoverable and show complete paths - #147

Merged
cpojer merged 9 commits into
nkzw-tech:mainfrom
AbdelrahmanHafez:feat/open-folder-entry-points
Jul 29, 2026
Merged

feat: make switching repositories discoverable and show complete paths#147
cpojer merged 9 commits into
nkzw-tech:mainfrom
AbdelrahmanHafez:feat/open-folder-entry-points

Conversation

@AbdelrahmanHafez

Copy link
Copy Markdown
Contributor

Based on feat/open-review-source-ui (#146), so the diff here includes those commits until it merges.

There was no way to switch repositories from within the app, only through the native menu bar. The repository label in the top bar also linked to the PR page, which the PR badge on the right already does. And paths rendered as fish-style abbreviations (~/d/w/blog instead of ~/dev/websites/blog), which reads like line noise if you don't live in fish.

This PR:

  • adds an Open Folder entry to the + menu and the command palette
  • makes the repository label plain text with the full path in its tooltip, instead of duplicating the PR link
  • shows complete repository paths everywhere, abbreviating only the home directory to ~

The top bar label keeps the repository name visible when space runs out: the head of the path shrinks with an ellipsis in the middle, the last segment never truncates. Paths outside the home directory stay absolute, and Windows paths keep their drive prefix.

image image

Created with Fable 5 xhigh, review iteration with Sol 5.6 xhigh. Built and tested and everything works as expected.

@cpojer

cpojer commented Jul 28, 2026

Copy link
Copy Markdown
Member

Thanks! Please rebase on main. I don't believe this should changed SharedWalkthroughApp (the web version) though? Because this would show local repo paths there in the wrong way. Please scope the changes only to the desktop app. Also I think extremely long repo names are clipped without an ellipsis as fixed-size tails cannot shrink. Maybe that could be fixed, too.

@AbdelrahmanHafez
AbdelrahmanHafez force-pushed the feat/open-folder-entry-points branch from 327e34a to 8374a00 Compare July 28, 2026 23:39
@AbdelrahmanHafez
AbdelrahmanHafez force-pushed the feat/open-folder-entry-points branch from 4d09b82 to c333d0c Compare July 29, 2026 07:19
@AbdelrahmanHafez

AbdelrahmanHafez commented Jul 29, 2026

Copy link
Copy Markdown
Contributor Author

Rebased on main. On the shared viewer, it was already showing local paths before this PR, just fish-abbreviated (~/d/codiff-web instead of ~/dev/codiff-web), this PR only makes them complete. I kept the change there so the web viewer and the desktop app stay consistent, but if you feel strongly about it, I'm happy to scope it back to the desktop.

Long repo names are fixed too. The tail span was flex: none so it could never shrink and just got clipped by the parent, now it's capped at the slot width and gets its own ellipsis once the head has fully collapsed. A normal repo name still never truncates while there's a head left to shrink.

image image image

@cpojer

cpojer commented Jul 29, 2026

Copy link
Copy Markdown
Member

Thank you

@cpojer
cpojer merged commit 955f6f4 into nkzw-tech:main Jul 29, 2026
1 check passed
@AbdelrahmanHafez
AbdelrahmanHafez deleted the feat/open-folder-entry-points branch July 29, 2026 18:50
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.

2 participants