Revert "PLT-395: Add syntax highlighting to Markdown code blocks"

web/react/components/user_settings/code_theme_chooser.jsx

@@ -1,55 +0,0 @@
-// Copyright (c) 2015 Mattermost, Inc. All Rights Reserved.
-// See License.txt for license information.
-
-var Constants = require('../../utils/constants.jsx');
-
-export default class CodeThemeChooser extends React.Component {
-    constructor(props) {
-        super(props);
-        this.state = {};
-    }
-    render() {
-        const theme = this.props.theme;
-
-        const premadeThemes = [];
-        for (const k in Constants.CODE_THEMES) {
-            if (Constants.CODE_THEMES.hasOwnProperty(k)) {
-                let activeClass = '';
-                if (k === theme.codeTheme) {
-                    activeClass = 'active';
-                }
-
-                premadeThemes.push(
-                    <div
-                        className='col-xs-6 col-sm-3 premade-themes'
-                        key={'premade-theme-key' + k}
-                    >
-                        <div
-                            className={activeClass}
-                            onClick={() => this.props.updateTheme(k)}
-                        >
-                            <label>
-                                <img
-                                    className='img-responsive'
-                                    src={'/static/images/themes/code_themes/' + k + '.png'}
-                                />
-                                <div className='theme-label'>{Constants.CODE_THEMES[k]}</div>
-                            </label>
-                        </div>
-                    </div>
-                );
-            }
-        }
-
-        return (
-            <div className='row'>
-                {premadeThemes}
-            </div>
-        );
-    }
-}
-
-CodeThemeChooser.propTypes = {
-    theme: React.PropTypes.object.isRequired,
-    updateTheme: React.PropTypes.func.isRequired
-};

web/react/components/user_settings/user_settings_appearance.jsx

@@ -7,7 +7,6 @@ var Utils = require('../../utils/utils.jsx');
 
 const CustomThemeChooser = require('./custom_theme_chooser.jsx');
 const PremadeThemeChooser = require('./premade_theme_chooser.jsx');
-const CodeThemeChooser = require('./code_theme_chooser.jsx');
 const AppDispatcher = require('../../dispatcher/app_dispatcher.jsx');
 const Constants = require('../../utils/constants.jsx');
 const ActionTypes = Constants.ActionTypes;
@@ -19,14 +18,12 @@ export default class UserSettingsAppearance extends React.Component {
         this.onChange = this.onChange.bind(this);
         this.submitTheme = this.submitTheme.bind(this);
         this.updateTheme = this.updateTheme.bind(this);
-        this.updateCodeTheme = this.updateCodeTheme.bind(this);
         this.handleClose = this.handleClose.bind(this);
         this.handleImportModal = this.handleImportModal.bind(this);
 
         this.state = this.getStateFromStores();
 
         this.originalTheme = this.state.theme;
-        this.originalCodeTheme = this.state.theme.codeTheme;
     }
     componentDidMount() {
         UserStore.addChangeListener(this.onChange);
@@ -61,10 +58,6 @@ export default class UserSettingsAppearance extends React.Component {
             type = 'custom';
         }
 
-        if (!theme.codeTheme) {
-            theme.codeTheme = Constants.DEFAULT_CODE_THEME;
-        }
-
         return {theme, type};
     }
     onChange() {
@@ -100,13 +93,6 @@ export default class UserSettingsAppearance extends React.Component {
         );
     }
     updateTheme(theme) {
-        theme.codeTheme = this.state.theme.codeTheme;
-        this.setState({theme});
-        Utils.applyTheme(theme);
-    }
-    updateCodeTheme(codeTheme) {
-        var theme = this.state.theme;
-        theme.codeTheme = codeTheme;
         this.setState({theme});
         Utils.applyTheme(theme);
     }
@@ -116,7 +102,6 @@ export default class UserSettingsAppearance extends React.Component {
     handleClose() {
         const state = this.getStateFromStores();
         state.serverError = null;
-        state.theme.codeTheme = this.originalCodeTheme;
 
         Utils.applyTheme(state.theme);
 
@@ -185,13 +170,7 @@ export default class UserSettingsAppearance extends React.Component {
                     </div>
                     {custom}
                     <hr />
-                    <strong className='radio'>{'Code Theme'}</strong>
+                                {serverError}
-                    <CodeThemeChooser
-                        theme={this.state.theme}
-                        updateTheme={this.updateCodeTheme}
-                    />
-                    <hr />
-                    {serverError}
                     <a
                         className='btn btn-sm btn-primary'
                         href='#'

web/react/package.json

@@ -6,7 +6,6 @@
     "autolinker": "0.18.1",
     "babel-runtime": "5.8.24",
     "flux": "2.1.1",
-    "highlight.js": "^8.9.1",
     "keymirror": "0.1.1",
     "marked": "0.3.5",
     "object-assign": "3.0.0",

web/react/utils/constants.jsx

@@ -302,13 +302,6 @@ module.exports = {
             uiName: 'Mention Highlight Link'
         }
     ],
-    CODE_THEMES: {
-        github: 'GitHub',
-        solarized_light: 'Solarized light',
-        monokai: 'Monokai',
-        solarized_dark: 'Solarized Dark'
-    },
-    DEFAULT_CODE_THEME: 'github',
     Preferences: {
         CATEGORY_DIRECT_CHANNEL_SHOW: 'direct_channel_show',
         CATEGORY_DISPLAY_SETTINGS: 'display_settings'
@@ -322,30 +315,5 @@ module.exports = {
         ENTER: 13,
         ESCAPE: 27,
         SPACE: 32
-    },
-    HighlightedLanguages: {
-        diff: 'Diff',
-        apache: 'Apache',
-        makefile: 'Makefile',
-        http: 'HTTP',
-        json: 'JSON',
-        markdown: 'Markdown',
-        javascript: 'JavaScript',
-        css: 'CSS',
-        nginx: 'nginx',
-        objectivec: 'Objective-C',
-        python: 'Python',
-        xml: 'XML',
-        perl: 'Perl',
-        bash: 'Bash',
-        php: 'PHP',
-        coffeescript: 'CoffeeScript',
-        cs: 'C#',
-        cpp: 'C++',
-        sql: 'SQL',
-        go: 'Go',
-        ruby: 'Ruby',
-        java: 'Java',
-        ini: 'ini'
     }
 };

web/react/utils/markdown.jsx

@@ -6,34 +6,6 @@ const Utils = require('./utils.jsx');
 
 const marked = require('marked');
 
-const highlightJs = require('highlight.js/lib/highlight.js');
-const highlightJsDiff = require('highlight.js/lib/languages/diff.js');
-const highlightJsApache = require('highlight.js/lib/languages/apache.js');
-const highlightJsMakefile = require('highlight.js/lib/languages/makefile.js');
-const highlightJsHttp = require('highlight.js/lib/languages/http.js');
-const highlightJsJson = require('highlight.js/lib/languages/json.js');
-const highlightJsMarkdown = require('highlight.js/lib/languages/markdown.js');
-const highlightJsJavascript = require('highlight.js/lib/languages/javascript.js');
-const highlightJsCss = require('highlight.js/lib/languages/css.js');
-const highlightJsNginx = require('highlight.js/lib/languages/nginx.js');
-const highlightJsObjectivec = require('highlight.js/lib/languages/objectivec.js');
-const highlightJsPython = require('highlight.js/lib/languages/python.js');
-const highlightJsXml = require('highlight.js/lib/languages/xml.js');
-const highlightJsPerl = require('highlight.js/lib/languages/perl.js');
-const highlightJsBash = require('highlight.js/lib/languages/bash.js');
-const highlightJsPhp = require('highlight.js/lib/languages/php.js');
-const highlightJsCoffeescript = require('highlight.js/lib/languages/coffeescript.js');
-const highlightJsCs = require('highlight.js/lib/languages/cs.js');
-const highlightJsCpp = require('highlight.js/lib/languages/cpp.js');
-const highlightJsSql = require('highlight.js/lib/languages/sql.js');
-const highlightJsGo = require('highlight.js/lib/languages/go.js');
-const highlightJsRuby = require('highlight.js/lib/languages/ruby.js');
-const highlightJsJava = require('highlight.js/lib/languages/java.js');
-const highlightJsIni = require('highlight.js/lib/languages/ini.js');
-
-const Constants = require('../utils/constants.jsx');
-const HighlightedLanguages = Constants.HighlightedLanguages;
-
 export class MattermostMarkdownRenderer extends marked.Renderer {
     constructor(options, formattingOptions = {}) {
         super(options);
@@ -43,43 +15,6 @@ export class MattermostMarkdownRenderer extends marked.Renderer {
         this.text = this.text.bind(this);
 
         this.formattingOptions = formattingOptions;
-
-        highlightJs.registerLanguage('diff', highlightJsDiff);
-        highlightJs.registerLanguage('apache', highlightJsApache);
-        highlightJs.registerLanguage('makefile', highlightJsMakefile);
-        highlightJs.registerLanguage('http', highlightJsHttp);
-        highlightJs.registerLanguage('json', highlightJsJson);
-        highlightJs.registerLanguage('markdown', highlightJsMarkdown);
-        highlightJs.registerLanguage('javascript', highlightJsJavascript);
-        highlightJs.registerLanguage('css', highlightJsCss);
-        highlightJs.registerLanguage('nginx', highlightJsNginx);
-        highlightJs.registerLanguage('objectivec', highlightJsObjectivec);
-        highlightJs.registerLanguage('python', highlightJsPython);
-        highlightJs.registerLanguage('xml', highlightJsXml);
-        highlightJs.registerLanguage('perl', highlightJsPerl);
-        highlightJs.registerLanguage('bash', highlightJsBash);
-        highlightJs.registerLanguage('php', highlightJsPhp);
-        highlightJs.registerLanguage('coffeescript', highlightJsCoffeescript);
-        highlightJs.registerLanguage('cs', highlightJsCs);
-        highlightJs.registerLanguage('cpp', highlightJsCpp);
-        highlightJs.registerLanguage('sql', highlightJsSql);
-        highlightJs.registerLanguage('go', highlightJsGo);
-        highlightJs.registerLanguage('ruby', highlightJsRuby);
-        highlightJs.registerLanguage('java', highlightJsJava);
-        highlightJs.registerLanguage('ini', highlightJsIni);
-    }
-
-    code(code, language) {
-        if (!language || highlightJs.listLanguages().indexOf(language) < 0) {
-            let parsed = super.code(code, language);
-            return '<code class="hljs">' + $(parsed).text() + '</code>';
-        }
-
-        let parsed = highlightJs.highlight(language, code);
-        return '<div class="post-body--code">' +
-            '<span class="post-body--code__language">' + HighlightedLanguages[language] + '</span>' +
-            '<code style="white-space: pre;" class="hljs">' + parsed.value + '</code>' +
-            '</div>';
     }
 
     br() {

web/react/utils/utils.jsx

@@ -408,11 +408,6 @@ export function toTitleCase(str) {
 }
 
 export function applyTheme(theme) {
-    if (!theme.codeTheme) {
-        theme.codeTheme = Constants.DEFAULT_CODE_THEME;
-    }
-    updateCodeTheme(theme.codeTheme);
-
     if (theme.sidebarBg) {
         changeCss('.sidebar--left, .settings-modal .settings-table .settings-links, .sidebar--menu', 'background:' + theme.sidebarBg, 1);
     }
@@ -599,27 +594,6 @@ export function rgb2hex(rgbIn) {
     return '#' + hex(rgb[1]) + hex(rgb[2]) + hex(rgb[3]);
 }
 
-export function updateCodeTheme(theme) {
-    const path = '/static/css/highlight/' + theme + '.css';
-    const $link = $('link.code_theme');
-    if (path !== $link.attr('href')) {
-        changeCss('code.hljs', 'visibility: hidden');
-        var xmlHTTP = new XMLHttpRequest();
-        xmlHTTP.open('GET', path, true);
-        xmlHTTP.onload = function onLoad() {
-            $link.attr('href', path);
-            if (isBrowserFirefox()) {
-                $link.one('load', () => {
-                    changeCss('code.hljs', 'visibility: visible');
-                });
-            } else {
-                changeCss('code.hljs', 'visibility: visible');
-            }
-        };
-        xmlHTTP.send();
-    }
-}
-
 export function placeCaretAtEnd(el) {
     el.focus();
     if (typeof window.getSelection != 'undefined' && typeof document.createRange != 'undefined') {

web/sass-files/sass/partials/_post.scss

@@ -466,22 +466,6 @@ body.ios {
 			white-space: nowrap;
 			cursor: pointer;
 		}
-		.post-body--code {
-			font-size: .97em;
-			position:relative;
-			.post-body--code__language {
-				position: absolute;
-				right: 0;
-				background: #fff;
-				cursor: default;
-				padding: 0.3em 0.5em 0.1em;
-				border-bottom-left-radius: 4px;
-				@include opacity(.3);
-			}
-			code {
-				white-space: pre;
-			}
-		}
 	}
 	.create-reply-form-wrap {
 		width: 100%;

web/static/css/highlight

@@ -1 +0,0 @@
-../../react/node_modules/highlight.js/styles/

web/templates/head.html

@@ -24,7 +24,6 @@
     <link rel="stylesheet" href="/static/css/bootstrap-colorpicker.min.css">
     <link rel="stylesheet" href="/static/css/styles.css">
     <link rel="stylesheet" href="/static/css/google-fonts.css">
-    <link rel="stylesheet" class="code_theme" href="">
 
     <link id="favicon" rel="icon" href="/static/images/favicon.ico" type="image/x-icon">
     <link rel="shortcut icon" href="/static/images/favicon.ico" type="image/x-icon">