fix(theme): make sure use default theme by default

User could use `theme="ide"` and get no theme on editor
JetBrains · Aug 3, 2018 · 5e6f265 · 5e6f265
1 parent b9239fd
commit 5e6f265
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/executable-code/index.js b/src/executable-code/index.js
@@ -65,7 +65,7 @@ export default class ExecutableCode {
     const indent = targetNode.hasAttribute(ATTRIBUTES.INDENT) ? parseInt(targetNode.getAttribute(ATTRIBUTES.INDENT)) : DEFAULT_INDENT;
     const from = targetNode.hasAttribute(ATTRIBUTES.FROM) ? parseInt(targetNode.getAttribute(ATTRIBUTES.FROM)) : null;
     const to = targetNode.hasAttribute(ATTRIBUTES.TO) ? parseInt(targetNode.getAttribute(ATTRIBUTES.TO)) : null;
-    const editorTheme = targetNode.hasAttribute(ATTRIBUTES.THEME) ? targetNode.getAttribute(ATTRIBUTES.THEME) : THEMES.DEFAULT;
+    const editorTheme = this.getTheme(targetNode);
     const args = targetNode.hasAttribute(ATTRIBUTES.ARGUMENTS) ? targetNode.getAttribute(ATTRIBUTES.ARGUMENTS) : "";
     const hiddenDependencies = this.getHiddenDependencies(targetNode);
     let targetPlatform = targetNode.getAttribute(ATTRIBUTES.PLATFORM);
@@ -150,6 +150,18 @@ export default class ExecutableCode {
       }, [])
   }
 
+  getTheme(targetNode){
+    const theme = targetNode.getAttribute(ATTRIBUTES.THEME);
+    switch (theme) {
+      case THEMES.DARCULA:
+        return THEMES.DARCULA;
+      case THEMES.IDEA:
+        return THEMES.IDEA;
+      default:
+        return THEMES.DEFAULT;
+    }
+  }
+
   getMode(targetNode) {
     const mode = targetNode.getAttribute(ATTRIBUTES.MODE);
     switch (mode) {

diff --git a/src/utils.js b/src/utils.js
@@ -8,6 +8,7 @@ import defaultConfig from './config';
  */
 export const THEMES = {
   DARCULA: "darcula",
+  IDEA: "idea",
   DEFAULT: "default"
 };
 /**