Fixes incorrect folder highlighting in webview file trees#4911
Conversation
🤖 Augment PR SummarySummary: Fixes incorrect folder highlighting in webview file trees by using each folder’s relative path as the node identifier. 🤖 Was this summary useful? React with 👍 or 👎 |
| branch: false, | ||
| expanded: true, | ||
| path: name, | ||
| path: relativePath, |
There was a problem hiding this comment.
relativePath from makeHierarchical is built using (...parts) => parts.join('/'), which results in top-level folder paths like /foo because the seed starts as an empty string. If any consumers expect folder path values to match file.path formatting (foo/bar without a leading slash), this could cause selection/lookups to behave unexpectedly.
Other Locations
src/webviews/apps/plus/patchDetails/components/gl-tree-base.ts:135
🤖 Was this useful? React with 👍 or 👎
e5621aa to
715107a
Compare
Problem
Selecting a folder in the webview file tree incorrectly highlights other folders with the same name, causing confusion in navigation and selection.
Solution
Benefit
Improves user experience and accuracy when working with nested folders in file trees within webview panels.
Fixes #4801