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

feat: syntax highlight #3273

Merged
merged 2 commits into from
Oct 26, 2023
Merged

feat: syntax highlight #3273

merged 2 commits into from
Oct 26, 2023

Conversation

anxolin
Copy link
Contributor

@anxolin anxolin commented Oct 26, 2023

Summary

Small PR to add syntax highlight. It also makes the modal a bit bigger so we can read well the snippet

Before:
image

After
image

I changed the command to start the configurator to be more in line with the project, and added it to the README

To Test

yarn start:widget

Check the code

@vercel
Copy link

vercel bot commented Oct 26, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
swap-dev ✅ Ready (Inspect) Visit Preview Oct 26, 2023 9:59am

🌃 Cosmos ↗︎

Copy link
Collaborator

@shoom3301 shoom3301 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect!

@anxolin anxolin requested a review from a team October 26, 2023 10:11
@socket-security
Copy link

New dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
@types/react-syntax-highlighter 15.5.9 None +0 195 kB types
react-syntax-highlighter 15.5.0 filesystem +13 9.38 MB simmerer

@anxolin anxolin merged commit 23ef99f into develop Oct 26, 2023
6 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Oct 26, 2023
@alfetopito alfetopito deleted the syntax-highlight branch October 26, 2023 17:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants