Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhanced [UI]: Copy Button for Improved Readability #436

Closed
wants to merge 2 commits into from

Conversation

SOUMITRO-SAHA
Copy link
Contributor

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

@hkirat
Copy link
Collaborator

hkirat commented May 19, 2024

Can u get rid of Copied ?

@SOUMITRO-SAHA
Copy link
Contributor Author

Should I replace it with a checkmark?

@hkirat
Copy link
Collaborator

hkirat commented May 19, 2024

yes

@SOUMITRO-SAHA
Copy link
Contributor Author

Done 👍

@devsargam devsargam closed this Aug 9, 2024
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.

bug: The copy button on CodeBlocks hides the code
3 participants