Skip to content

refactor: consolidate duplicated copy/clipboard icons and operations#537

Open
Green00101 wants to merge 2 commits intobacknotprop:mainfrom
Green00101:refactor/consolidate-copy-clipboard
Open

refactor: consolidate duplicated copy/clipboard icons and operations#537
Green00101 wants to merge 2 commits intobacknotprop:mainfrom
Green00101:refactor/consolidate-copy-clipboard

Conversation

@Green00101
Copy link
Copy Markdown

Summary

This PR consolidates duplicated copy/clipboard UI and logic into shared UI utilities.

Changes

  • add shared CopyIcon and CheckIcon in packages/ui/components/icons/copyIcons.tsx
  • add shared useCopyToClipboard hook in packages/ui/hooks/useCopyToClipboard.ts
  • move the canonical CopyButton implementation to packages/ui/components/CopyButton.tsx
  • keep packages/review-editor/components/CopyButton.tsx as a re-export for backward compatibility
  • replace duplicated inline icons and clipboard logic in affected editor, review-editor, and ui components

Notes

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.

refactor: consolidate duplicated copy/clipboard icons and operations

1 participant