Skip to content

ref(scraps): Extract clipboard button with app feedback - #129520

Open
natemoo-re wants to merge 4 commits into
masterfrom
ref/de-1742-clipboard
Open

natemoo-re wants to merge 4 commits into
masterfrom
ref/de-1742-clipboard

Conversation

@natemoo-re

@natemoo-re natemoo-re commented Nov 2, 2026 •

Copy link
Copy Markdown
Member

Moves CopyToClipboardButton into @sentry/scraps/clipboard. Feedback uses the shared Toast and translation APIs, with per-instance success and error message overrides. App callers import the package directly.

Component tests run from the package; application feedback coverage remains in the app. No visual changes intended.

closes DE-1742

Keep clipboard writes portable and inject application notifications through a feedback context. Preserve the existing app wrapper and successful-copy callbacks.

Refs DE-1742
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 2, 2026
@github-actions

github-actions Bot commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

@linear-code

linear-code Bot commented Nov 2, 2026

Copy link
Copy Markdown

DE-1742

@natemoo-re
natemoo-re marked this pull request as ready for review November 2, 2026 19:43
@natemoo-re
natemoo-re requested review from a team as code owners November 2, 2026 19:43

@TkDodo TkDodo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not approving because I think the Provider is wrong

Keep clipboard feedback inside Scraps and allow per-instance messages. Remove redundant feedback providers and migrate app callers to the public clipboard entry point while integrating current master.

This branch was successfully deployed

1 active deployment
Preview — 33a0344b Deployed Nov 3, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants