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.

Sign up for GitHub

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

Jump to bottom

Add copy to clipboard action for code snippets at sample section. #251

Merged
merged 5 commits into from
Dec 4, 2022
Merged

Add copy to clipboard action for code snippets at sample section. #251

merged 5 commits into from
Dec 4, 2022

Conversation

brkcrdk
Copy link
Contributor

@brkcrdk brkcrdk commented Dec 2, 2022

  • Add copy to clipboard action at Sample section.

Ekran Resmi 2022-12-02 22 01 51
Ekran Resmi 2022-12-02 22 06 14
Ekran Resmi 2022-12-02 22 02 34

@aykutkardas aykutkardas self-requested a review December 2, 2022 19:50
@brkcrdk
Copy link
Contributor Author

brkcrdk commented Dec 2, 2022

With new updates:

  • Icon placement is changed
  • Added react-syntax-highlights type definitions to get better suggestions for library usage and typesafety.
  • Example of updated ui changes:
Ekran.Kaydi.2022-12-02.23.55.40.mov
Ekran.Kaydi.2022-12-02.23.50.17.mov

@aykutkardas aykutkardas mentioned this pull request Dec 3, 2022
Closed
@aykutkardas aykutkardas closed this Dec 3, 2022
@aykutkardas aykutkardas reopened this Dec 3, 2022
@aykutkardas aykutkardas merged commit 6f5714b into aykutkardas:main Dec 4, 2022
@brkcrdk brkcrdk deleted the copy-code-snippet branch December 5, 2022 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aykutkardas aykutkardas aykutkardas approved these changes

Assignees

@brkcrdk brkcrdk

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@brkcrdk @aykutkardas