fix: remove glow theme (#545)

* fix: remove glow theme

* chore: set active node colour to match zelos glow colour

Author: rachel-fenichel

src/index.ts

@@ -20,12 +20,6 @@ export class KeyboardNavigation {
   /** Cursor for the main workspace. */
   private cursor: Blockly.LineCursor;
 
-  /**
-   * These fields are used to preserve the workspace's initial state to restore
-   * it when/if keyboard navigation is disabled.
-   */
-  private originalTheme: Blockly.Theme;
-
   /**
    * Input mode tracking.
    */
@@ -62,9 +56,6 @@ export class KeyboardNavigation {
     this.navigationController.enable(workspace);
     this.inputModeTracker = new InputModeTracker(workspace);
 
-    this.originalTheme = workspace.getTheme();
-    this.setGlowTheme();
-
     this.cursor = new Blockly.LineCursor(workspace);
 
     // Add the event listener to enable disabled blocks on drag.
@@ -132,9 +123,6 @@ export class KeyboardNavigation {
 
     // Remove the event listener that enables blocks on drag
     this.workspace.removeChangeListener(enableBlocksOnDrag);
-
-    this.workspace.setTheme(this.originalTheme);
-
     this.navigationController.dispose();
     this.inputModeTracker.dispose();
   }
@@ -145,19 +133,4 @@ export class KeyboardNavigation {
   toggleShortcutDialog(): void {
     this.navigationController.shortcutDialog.toggle(this.workspace);
   }
-
-  /**
-   * Update the theme to match the selected glow colour to the cursor
-   * colour.
-   */
-  setGlowTheme() {
-    const newTheme = Blockly.Theme.defineTheme('zelosDerived', {
-      name: 'zelosDerived',
-      base: Blockly.Themes.Zelos,
-      componentStyles: {
-        selectedGlowColour: '#ffa200',
-      },
-    });
-    this.workspace.setTheme(newTheme);
-  }
 }

test/index.html

@@ -91,7 +91,7 @@
       }
 
       html {
-        --blockly-active-node-color: #ffa200;
+        --blockly-active-node-color: #fff200;
         --blockly-active-tree-color: #60a5fa;
         --blockly-selection-width: 3px;
       }