Fix for #7229 Fix/project sidebar folder name #7232
Merged
+4
−8
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix for #7229 : Project sidebar displays full paths instead of folder names on Windows
Problem:
On Windows, the sidebar showed full paths like C:\Users{name}...\project-name instead of just project-name
The tooltip on hover also displayed the full path
Root cause:
getFilename() utility only split by /, not handling Windows \ separator
Tooltip directly used worktree instead of extracting the folder name
Duplicate getFilename function in dialog-edit-project.tsx
Changes:
Updated getFilename() in packages/util/src/path.ts to split by both / and
Changed tooltip in packages/app/src/pages/layout.tsx to show folder name
Removed duplicate function, using imported utility in packages/app/src/components/dialog-edit-project.tsx
Before:

After:

Note: this is a redo of #7230 as I messed up and included commits from an unmerged PR in it and so closed and redid it here