-
Notifications
You must be signed in to change notification settings - Fork 2.8k
feat: add file path tooltips with centralized PathTooltip component #9030
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add PathTooltip component that wraps StandardTooltip with proper styling - Use maxWidth='min(300px,100vw)' via inline style to override defaults - Apply '[text-wrap:wrap]' class to override text-balance behavior - Add formatPathTooltip helper for consistent path content formatting - Update CodeAccordian, ChatRow, and BatchFilePermission to use PathTooltip - Centralizes tooltip behavior and reduces duplication Supersedes PR #8797.
Contributor
|
Review complete. No issues found - the implementation is clean, type-safe, and correctly adds tooltip functionality to truncated file paths. Mention @roomote in a comment to trigger your PR Fixer agent and make changes to this pull request. |
7 tasks
mrubens
approved these changes
Nov 5, 2025
mini2s
added a commit
to zgsm-ai/costrict
that referenced
this pull request
Nov 6, 2025
* feat: add UsageStats schema and type (RooCodeInc#8441) feat: add UsageStats schema and type to cloud.ts Co-authored-by: Roo Code <[email protected]> * Release: v1.80.0 (RooCodeInc#8442) chore: bump version to v1.80.0 * feat: add new DeepSeek and GLM models with detailed descriptions to the Chutes provider (RooCodeInc#8467) * Deprecate free grok 4 fast (RooCodeInc#8481) Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com> * fix: improve save button activation in prompts settings (RooCodeInc#5780) (RooCodeInc#8267) Co-authored-by: MuriloFP <[email protected]> Co-authored-by: Roo Code <[email protected]> Co-authored-by: daniel-lxs <[email protected]> * fix: properly reset cost limit tracking when user clicks "Reset and Continue" (RooCodeInc#6890) Co-authored-by: Roo Code <[email protected]> Co-authored-by: daniel-lxs <[email protected]> * chore(deps): update dependency vite to v6.3.6 [security] (RooCodeInc#7838) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * chore(deps): update dependency glob to v11.0.3 (RooCodeInc#7767) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * chore: add changeset for v3.28.15 (RooCodeInc#8491) * Changeset version bump (RooCodeInc#8492) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Matt Rubens <[email protected]> * Clamp GPT-5 max output tokens to 20% of context window (
Related GitHub Issue
Closes: #8278
Description
This PR adds tooltips to display full file paths on truncated filenames in file editing/reading notices throughout the chat interface.
Implementation details:
PathTooltipcomponent that wrapsStandardTooltipwith appropriate stylingmaxWidth="min(300px,100vw)"via inline style to reliably override defaults[text-wrap:wrap]class to override text-balance behaviorformatPathTooltiphelper for consistent path content formattingCodeAccordian.tsx,ChatRow.tsx, andBatchFilePermission.tsxto usePathTooltipKey design choices:
max-w-[300px]inTooltipContent[text-wrap:wrap]arbitrary property to reliably override the defaulttext-balanceCSSTesting:
Test Procedure
Manual testing steps:
Test truncated paths show tooltips:
src/components/chat/file/with/deeply/nested/structure.tsx)Test all tooltip locations:
Pre-Submission Checklist
Screenshot
Supersedes PR #8797.
Important
Introduces
PathTooltipcomponent to centralize and standardize file path tooltips across the application, reducing code duplication and ensuring consistent behavior.PathTooltipinPathTooltip.tsxfor displaying full file paths.PathTooltipinBatchFilePermission.tsx,ChatRow.tsx, andCodeAccordian.tsx.formatPathTooltipinformatPathTooltip.tsfor consistent path formatting.This description was created by
for fcb9dbe. You can customize this summary. It will automatically update as commits are pushed.