daily-code icon indicating copy to clipboard operation
daily-code copied to clipboard

Published 20 hours ago • verified publisher icon code100x

Enhanced [UI]: Copy Button for Improved Readability

Open SOUMITRO-SAHA opened this issue 1 year ago • 4 comments

PR Fixes:

  1. Fixed the Copy Button.
  2. Previously, the button hid the text behind it. This issue has been resolved with a more professional approach.
  3. Converted the component to a client component to ensure it uses React Hooks correctly. Although no errors are currently shown since it's used within a client component, adding "use client" at the top ensures it won't cause issues if used in a server-side component in the future.
Before

image

After

image

Resolves #425

Checklist before requesting a review

  • [ ] I have performed a self-review of my code
  • [ ] I assure there is no similar/duplicate pull request regarding same issue

SOUMITRO-SAHA avatar May 19 '24 15:05 SOUMITRO-SAHA

Can u get rid of Copied ?

hkirat avatar May 19 '24 17:05 hkirat

Should I replace it with a checkmark?

SOUMITRO-SAHA avatar May 19 '24 18:05 SOUMITRO-SAHA

yes

hkirat avatar May 19 '24 18:05 hkirat

Done 👍

SOUMITRO-SAHA avatar May 19 '24 18:05 SOUMITRO-SAHA