-
Notifications
You must be signed in to change notification settings - Fork 8
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
Added copy code feature for mdx code blocks #330
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@zahidalidev I have reviewed the PR. The table issue seems to be fixed, but the copy code icons seem to be off in alignment in some places, especially with lengthy lines of code. The screenshot shows the overlap. |
@MaedahBatool I've pushed the changes and fixed the feedback points. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Pull Request approval
Although pull request approval is not enforced for this repository in order to reduce friction, merging without a review will generate a ticket for the docs team to review your changes. So if possible, have your pull request approved before merging.