Skip to content

[Docs] Switch to codemirror's JSX mode#6898

Merged
zpao merged 1 commit intofacebook:masterfrom
keyz:site-codemirror-jsx
May 27, 2016
Merged

[Docs] Switch to codemirror's JSX mode#6898
zpao merged 1 commit intofacebook:masterfrom
keyz:site-codemirror-jsx

Conversation

@keyz
Copy link
Copy Markdown
Contributor

@keyz keyz commented May 26, 2016

Fixes #6894. I upgraded codemirror from 3.20.1 to 5.15.2 and switched to their JSX mode. Also I tweaked the theme a little bit since there are some code highlighting differences between v3 and v5.

This is a huge diff due to changing codemirror's source files, but the actual changes are just docs/_css/_solarized.scss, docs/_js/live_editor.js, and docs/_layouts/default.html.

Before:
screen shot 2016-05-26 at 4 25 07 pm

After:
screen shot 2016-05-26 at 4 25 52 pm

@keyz keyz changed the title [Docs] Switched to codemirror's JSX mode [Docs] Switch to codemirror's JSX mode May 26, 2016
@zpao
Copy link
Copy Markdown
Member

zpao commented May 27, 2016

lgtm, thanks!

@zpao zpao merged commit c8bab76 into facebook:master May 27, 2016
zpao pushed a commit that referenced this pull request May 31, 2016
@keyz keyz deleted the site-codemirror-jsx branch June 3, 2016 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants