Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add highlighting of user solutions. #226

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

nbeloglazov commented Aug 28, 2012

See #208.
I've used Ace for highlighting because it's already being used as editor. Though I had to update ace.js and clojure-mode.js to latest ones because static-highlight.js didn't work with old ones.

Owner

amalloy commented Aug 28, 2012

We're already using syntax-highlighter to display your own solution to you on the "share" page - is there a reason to use ace rather than SH?

Contributor

nbeloglazov commented Aug 28, 2012

Didn't know about share page and it's highlighting. I'll replace my Ace with SH so it will be less painful than using Ace for highlighting only.

Contributor

nbeloglazov commented Aug 29, 2012

@amalloy I've updated pull request. Can you take a look?

Owner

amalloy commented Aug 30, 2012

Thanks! I've rebased this to 0467f47 so as not to include the useless Ace commit/merge.

@amalloy amalloy closed this Aug 30, 2012

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