Skip to content

feat: clickable text and tooltip removal on CopyableText#30

Merged
cpascariello merged 2 commits intomainfrom
feature/copyable-text-enhancement
Mar 13, 2026
Merged

feat: clickable text and tooltip removal on CopyableText#30
cpascariello merged 2 commits intomainfrom
feature/copyable-text-enhancement

Conversation

@cpascariello
Copy link
Copy Markdown
Collaborator

Summary

  • Truncated text renders as an <a> tag when href is provided, making it clickable (opens in new tab)
  • Removed Tooltip wrapper entirely — the copy button already provides full text access
  • Dropped @radix-ui/react-tooltip dependency from CopyableText's render tree

Test plan

  • Existing tests pass (255/255)
  • New test verifies truncated text is a clickable link when href is set
  • Lint, typecheck, build all clean

…ve tooltip

- Render truncated text as an <a> when href exists instead of plain <span>
- Remove Tooltip wrapper entirely — copy button provides full text access
- Drop @radix-ui/react-tooltip dependency from CopyableText
@cpascariello cpascariello merged commit 6286e22 into main Mar 13, 2026
1 check passed
@cpascariello cpascariello deleted the feature/copyable-text-enhancement branch March 13, 2026 11:21
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.

1 participant