fix(desktop): add Copy Link action for chat links#1835
fix(desktop): add Copy Link action for chat links#1835Klemencina wants to merge 1 commit intopingdotgg:mainfrom
Conversation
- Show a Copy Link item for safe external URLs - Copy the clicked link URL to the clipboard
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ApprovabilityVerdict: Approved Simple UI enhancement adding a "Copy Link" context menu option for links. The change is self-contained, follows existing patterns, uses existing URL validation, and only adds standard clipboard functionality with no security or behavioral concerns. You can customize Macroscope's approvability policy. Learn more. |
What Changed
Adds a desktop-only
Copy Linkaction to the native context menu for safehttp/httpslinks in chat output.When right-clicking a chat link in the desktop app, the context menu now shows
Copy Link, which copies the clicked URL to the clipboard.Why
Fixes the missing link-copy action reported in #677.
This is intentionally narrower than #772. It only addresses the missing copy behavior and does not add
Open Linkor any other link actions.UI Changes
Before:
After:
http/httpschat link showsCopy LinkChecklist
Note
Add 'Copy Link' context menu action for chat links in the desktop app
Adds a 'Copy Link' item to the right-click context menu in main.ts when the clicked target has a URL that passes
getSafeExternalUrl. The URL is written to the system clipboard using Electron'sclipboardmodule, followed by a separator in the menu.Macroscope summarized b003d3a.