Guide for adding language syntax highlighting #24

Closed
wants to merge 4 commits into
from

Conversation

Projects
None yet
3 participants
@0X1A

0X1A commented Dec 7, 2013

No description provided.

0X1A added some commits Dec 7, 2013

Added guide for adding language support to prism.js
Signed-off-by: Alberto Corona <albcoron@gmail.com>
Sectioned guide
Signed-off-by: Alberto Corona <albcoron@gmail.com>
@oswaldoacauan

This comment has been minimized.

Show comment
Hide comment
@oswaldoacauan

oswaldoacauan Dec 8, 2013

Owner

WOW! Really nice!

You dont need to "download our own version" of PrismJS, since bower already download it for you, you just need to link these components on your build script. Take a look at /src/assets/_components/prismjs/components all languages are listed there :)

Instead of the copy you can use the new dryrun grunt task #23

Owner

oswaldoacauan commented Dec 8, 2013

WOW! Really nice!

You dont need to "download our own version" of PrismJS, since bower already download it for you, you just need to link these components on your build script. Take a look at /src/assets/_components/prismjs/components all languages are listed there :)

Instead of the copy you can use the new dryrun grunt task #23

@0X1A

This comment has been minimized.

Show comment
Hide comment
@0X1A

0X1A Dec 8, 2013

So the bower download of prism already includes the other languages? And dryrun is used to copy the new build to the root directory?

0X1A commented Dec 8, 2013

So the bower download of prism already includes the other languages? And dryrun is used to copy the new build to the root directory?

@oswaldoacauan

This comment has been minimized.

Show comment
Hide comment
@oswaldoacauan

oswaldoacauan Dec 9, 2013

Owner

@0X1A Exactly. To add more languages your only need to link them (prismjs components) after the prism.js lib on <!-- build:js... --> and run a grunt dryrun.

Owner

oswaldoacauan commented Dec 9, 2013

@0X1A Exactly. To add more languages your only need to link them (prismjs components) after the prism.js lib on <!-- build:js... --> and run a grunt dryrun.

Updated PRISM.md to include dryrun
Signed-off-by: Alberto Corona <albcoron@gmail.com>
@0X1A

This comment has been minimized.

Show comment
Hide comment
@0X1A

0X1A Dec 9, 2013

Alright, updated it to include dryrun. Let me know how that looks
Edit: Can't seem to get this to work on v1.2.1

0X1A commented Dec 9, 2013

Alright, updated it to include dryrun. Let me know how that looks
Edit: Can't seem to get this to work on v1.2.1

@digitaljhelms

This comment has been minimized.

Show comment
Hide comment
@digitaljhelms

digitaljhelms Dec 9, 2013

Contributor

FYI, rather than using a fork of Prism, I changed my fork of ghostium to use the authoritative/official fork; possibly something you want to consider upstream.

digitaljhelms/ghostium@a0db096

LMK if you want a PR.

PR issued: #27

Contributor

digitaljhelms commented Dec 9, 2013

FYI, rather than using a fork of Prism, I changed my fork of ghostium to use the authoritative/official fork; possibly something you want to consider upstream.

digitaljhelms/ghostium@a0db096

LMK if you want a PR.

PR issued: #27

@0X1A 0X1A closed this Apr 19, 2014

@andschwa andschwa referenced this pull request Apr 26, 2014

Closed

Edit src/stuff or ./stuff? #89

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment