scriptoriumadmin/Roo-Code
1
0
Fork 0
mirror of https://github.com/RooVetGit/Roo-Code.git synced 2026-10-10 03:28:03 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix Copy button logic (#4458)

Browse source 
* refactor: streamline text copying by removing history highlight spans

* refactor: improve regex for stripping history highlight spans to handle all content types

---------

Co-authored-by: Daniel Riccio <ricciodaniel98@gmail.com>
This commit is contained in:
Sam Hoang Van 2025-06-09 21:40:31 +07:00 • committed by GitHub
parent 360b0e7336
commit 19108f7136
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 16 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
webview-ui/src/components/history/CopyButton.tsx
View file

@ -10,6 +10,19 @@ type CopyButtonProps = {
className?: string
}
/**
* Strips only history highlight spans from text while preserving other HTML
* Targets: <span class="history-item-highlight">content</span>
* @param text - Text that may contain highlight spans
* @returns Text with highlight spans removed but content preserved
*/
const stripHistoryHighlightSpans = (text: string): string => {
// Match opening tag, capture content until closing tag
// The [\s\S]*? pattern matches any character (including newlines) non-greedily,
// which properly handles content with < characters
return text.replace(/<span\s+class="history-item-highlight">([\s\S]*?)<\/span>/g, "$1")
}
export const CopyButton = ({ itemTask, className }: CopyButtonProps) => {
const { isCopied, copy } = useClipboard()
const { t } = useAppTranslation()
@ -17,12 +30,10 @@ export const CopyButton = ({ itemTask, className }: CopyButtonProps) => {
const onCopy = useCallback(
(e: React.MouseEvent) => {
e.stopPropagation()
const tempDiv = document.createElement("div")
tempDiv.innerHTML = itemTask
const text = tempDiv.textContent || tempDiv.innerText || ""
if (!isCopied) {
copy(text)
// Strip only history highlight spans before copying to clipboard
const cleanText = stripHistoryHighlightSpans(itemTask)
copy(cleanText)
}
},
[isCopied, copy, itemTask],