feat/agent-filename#1595
Conversation
|
@4pmtong |
|
@Wendong-Fan |
|
Hi @ultrahighsuper Thanks for the contribution, this is a useful improvement and the underlying issue (long filenames getting truncated with no way to see the full name) is a real one. One consideration before this is ready to merge: right now TooltipSimple is applied to every file/folder row unconditionally, regardless of whether the name is actually truncated. This means short names that already fit fine will also pop up a tooltip on hover, which adds unnecessary noise and hurts the UX rather than helping it. Could you scope the tooltip so it only shows for entries whose name is actually cut off? Also, could you pull the latest main and resolve conflicts? We shipped a release last week with a fairly significant refactor of this component, so the current diff no longer applies cleanly against the latest version. Once that's updated, feel free to re-request review and tag me — happy to take another look. |
2 participants
Related Issue
Description
In the Agent Folder panel, long filenames are truncated with
truncate, making it hard to see the full name. This change adds a hover tooltip that displays the full file/folder name so users can quickly identify long entries.Implementation:
src/components/Folder/index.tsx(FileTree) to wrap each file/folder row button withTooltipSimpleusingcontent={child.name}.Testing Evidence (REQUIRED)
What is the purpose of this pull request?
Contribution Guidelines Acknowledgement