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

UG: copyedits of Prism section #1148

Merged
merged 2 commits into from
Aug 15, 2022
Merged

UG: copyedits of Prism section #1148

merged 2 commits into from
Aug 15, 2022

Conversation

chalin
Copy link
Collaborator

@chalin chalin commented Aug 9, 2022

```none
https://prismjs.com/download.html#themes=prism&languages=markup+css+clike+javascript+bash+c+csharp+cpp+go+java+markdown+python+scss+sql+toml+yaml&plugins=toolbar+copy-to-clipboard
```
https://prismjs.com/download.html#themes=prism&languages=markup+css+clike+javascript+bash+c+csharp+cpp+go+java+markdown+python+scss+sql+toml+yaml&plugins=toolbar+copy-to-clipboard
Copy link
Collaborator

Choose a reason for hiding this comment

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

This very long URL is overspilling the text area.
image

Is there a better way to convey this information (either just the list of languages that we support out of the box, or a different format for the URL)?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed. PTAL

@chalin chalin requested a review from LisaFC August 15, 2022 17:22
@LisaFC
Copy link
Collaborator

LisaFC commented Aug 15, 2022

Much better!

@chalin chalin merged commit bec5869 into main Aug 15, 2022
@chalin chalin deleted the chalin-patch-1 branch August 15, 2022 17:37
fekete-robert pushed a commit to fekete-robert/docsy that referenced this pull request Sep 13, 2022
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.

None yet

2 participants