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

Convert Google Prettify.js Links to CDN Links #116

Open
wants to merge 4 commits into
base: 2.0-dev
Choose a base branch
from

Conversation

spreadred
Copy link

Convert Google code-prettify links to use CDN and the new preferred
method of loading: run_prettify.js which no longer requires separate
includes of prettify.css

Addresses issue #115

Convert Google code-prettify links to use CDN and the new preferred
method of loading: run_prettify.js which no longer requires separate
includes of prettify.css
Copy link
Contributor

@codewithtyler codewithtyler left a comment

Choose a reason for hiding this comment

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

Is this working without prettify.css?

@spreadred
Copy link
Author

Yes. Here a link to a jsfiddle which simply includes run_prettify.js without the CSS.

kaptainkommie and others added 3 commits March 15, 2017 15:52
Added instructions on how to properly initialize multiple editors on the same page. Proper initialization and usage resolves steveathon#117 steveathon#110 steveathon#31
Added Multiple Editors Instructions
@spreadred
Copy link
Author

Looks like I messed up my PR. The only intended commit for this PR is the first commit on March 14 relating to the addition of the CDN links in the examples.

@steveathon
Copy link
Owner

@kaptainkommie did you want to cancel and re-open this PR with the cherry picked changes? Or go ahead with it as it is?

@yarkm13
Copy link

yarkm13 commented May 21, 2022

According to repo info rawgit.com was turned off October 2019.

It's possible to use this CDN now https://cdn.jsdelivr.net/gh/google/code-prettify@master/loader/run_prettify.js

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.

4 participants