Skip to content
This repository was archived by the owner on Feb 25, 2026. It is now read-only.

fix(ui): remove redundant copy button from TextPartDisplay #453

Closed
haliliceylan wants to merge 3 commits into from
Closed

fix(ui): remove redundant copy button from TextPartDisplay #453

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
e3b9da2
fix(ui): remove redundant copy button from TextPartDisplay
haliliceylan Feb 18, 2026
c2d977e
refactor(ui): remove unused text part copy wrapper styles
haliliceylan Feb 18, 2026
a668143
Merge branch 'dev' into dev
catrielmuller Feb 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 0 additions & 18 deletions packages/ui/src/components/message-part.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -114,24 +114,6 @@
[data-component="text-part"] {
width: 100%;

[data-slot="text-part-body"] {
position: relative;
margin-top: 32px;
}

[data-slot="text-part-copy-wrapper"] {
position: absolute;
top: -28px;
right: 8px;
opacity: 0;
transition: opacity 0.15s ease;
z-index: 1;
}

[data-slot="text-part-body"]:hover [data-slot="text-part-copy-wrapper"] {
opacity: 1;
}

[data-component="markdown"] {
margin-top: 0;
font-size: var(--font-size-base);
Expand Down
30 changes: 1 addition & 29 deletions packages/ui/src/components/message-part.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -668,42 +668,14 @@ PART_MAPPING["tool"] = function ToolPartDisplay(props) {

PART_MAPPING["text"] = function TextPartDisplay(props) {
const data = useData()
const i18n = useI18n()
const part = props.part as TextPart
const displayText = () => relativizeProjectPaths((part.text ?? "").trim(), data.directory)
const throttledText = createThrottledValue(displayText)
const [copied, setCopied] = createSignal(false)

const handleCopy = async () => {
const content = displayText()
if (!content) return
await navigator.clipboard.writeText(content)
setCopied(true)
setTimeout(() => setCopied(false), 2000)
}

return (
<Show when={throttledText()}>
<div data-component="text-part">
<div data-slot="text-part-body">
<Markdown text={throttledText()} cacheKey={part.id} />
<div data-slot="text-part-copy-wrapper">
<Tooltip
value={copied() ? i18n.t("ui.message.copied") : i18n.t("ui.message.copy")}
placement="top"
gutter={8}
>
<IconButton
icon={copied() ? "check" : "copy"}
size="small"
variant="secondary"
onMouseDown={(e) => e.preventDefault()}
onClick={handleCopy}
aria-label={copied() ? i18n.t("ui.message.copied") : i18n.t("ui.message.copy")}
/>
</Tooltip>
</div>
</div>
<Markdown text={throttledText()} cacheKey={part.id} />
</div>
</Show>
)
Expand Down