Skip to content

fix: show copy failure state#97

Open
omribz156 wants to merge 2 commits into
MyNameIsEdi:mainfrom
omribz156:codex/copy-button-error-state
Open

fix: show copy failure state#97
omribz156 wants to merge 2 commits into
MyNameIsEdi:mainfrom
omribz156:codex/copy-button-error-state

Conversation

@omribz156
Copy link
Copy Markdown
Contributor

@omribz156 omribz156 commented May 21, 2026

Summary:

  • Catch rejected clipboard writes in CodeSnippet
  • Show a translated failure state instead of Copied! when copying fails
  • Add English and Hebrew copy-failure labels

Verification:

  • npm ci --ignore-scripts --no-audit --no-fund
  • npm run ui:build
  • npx prettier --check ui/src/components/CodeSnippet.tsx ui/src/locales/en.json ui/src/locales/he.json
  • C:\Program Files\Git\bin\bash.exe scripts/check-docs-updated.sh
  • git diff --check

Note: npx tsc --project ui/tsconfig.json --noEmit --pretty false still fails on existing unrelated errors in AppShell.tsx and PlaywrightDashboard.tsx. GitHub Lint also fails on existing unused symbols in PlaywrightDashboard.tsx. The patch-specific missing icon import from the first local run was fixed, and the Vite build plus docs gate now pass.

This was implemented with Codex assistance and manually reviewed to keep the fix focused.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 23, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 81bb332f-4aab-4b33-bfe1-4727b234daf4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

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