From 45dc8a520f559dd5470a4367d7ba92e338248b19 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?L=C3=A1szl=C3=B3=20Monda?= <laci@monda.hu>
Date: Wed, 30 Mar 2016 17:55:10 +0200
Subject: [PATCH] Make layer toggleable.

---
 .../config-items/SwitchKeymapAction.ts        | 23 ++++++++++++++++---
 config-serializer/serializeConfig.ts          | 23 -------------------
 2 files changed, 20 insertions(+), 26 deletions(-)

diff --git a/config-serializer/config-items/SwitchKeymapAction.ts b/config-serializer/config-items/SwitchKeymapAction.ts
index 353b87e8..e1e3b1b2 100644
--- a/config-serializer/config-items/SwitchKeymapAction.ts
+++ b/config-serializer/config-items/SwitchKeymapAction.ts
@@ -1,8 +1,17 @@
+enum Layer {
+    mod,
+    fn,
+    mouse
+}
+
 class SwitchLayerAction extends KeyAction implements Serializable<SwitchLayerAction> {
 
     static keyActionTypeString = 'switchLayer';
+    static toggleFlag = 0x80;
 
-    private _layer: number;
+    isLayerToggleable: boolean;
+
+    private _layer: Layer;
 
     get layer(): number {
         return this._layer;
@@ -15,27 +24,35 @@ class SwitchLayerAction extends KeyAction implements Serializable<SwitchLayerAct
         this._layer = value;
     }
 
+    getToggleFlag() {
+        return this.isLayerToggleable ? SwitchLayerAction.toggleFlag : 0;
+    }
+
     fromJsObject(jsObject: any): SwitchLayerAction {
         this.assertKeyActionType(jsObject, SwitchLayerAction.keyActionTypeString, 'SwitchLayerAction');
         this.layer = jsObject.keymapId;
+        this.isLayerToggleable = jsObject.toggle;
         return this;
     }
 
     fromBinary(buffer: UhkBuffer): SwitchLayerAction {
         this.readAndAssertKeyActionId(buffer, KeyActionId.SwitchLayerAction, 'SwitchLayerAction');
         this.layer = buffer.readUInt8();
+        this.isLayerToggleable = (this.layer & SwitchLayerAction.toggleFlag) !== 0;
+        this.layer &= ~SwitchLayerAction.toggleFlag; // Clear toggle bit.
         return this;
     }
 
     toJsObject(): any {
         return {
             keyActionType: SwitchLayerAction.keyActionTypeString,
-            layer: this.layer
+            layer: this.layer,
+            toggle: this.isLayerToggleable
         };
     }
 
     toBinary(buffer: UhkBuffer) {
         buffer.writeUInt8(KeyActionId.SwitchLayerAction);
-        buffer.writeUInt8(this.layer);
+        buffer.writeUInt8(this.layer | this.getToggleFlag());
     }
 }
diff --git a/config-serializer/serializeConfig.ts b/config-serializer/serializeConfig.ts
index 563e6d06..8097e6e2 100644
--- a/config-serializer/serializeConfig.ts
+++ b/config-serializer/serializeConfig.ts
@@ -9,27 +9,4 @@ let writer = new UhkBuffer();
 let uhkConfig = JSON.parse(fs.readFileSync('uhk-config.json'));
 let keyActions = uhkConfig.keymaps[0].layers[0].modules[0].keyActions;
 
-let SWITCH_LAYER_MOD    = 0;
-let SWITCH_LAYER_FN     = 1;
-let SWITCH_LAYER_MOUSE  = 2;
-let SWITCH_LAYER_TOGGLE = 0x80;
-
-function serializeKeyAction(keyAction) {
-    switch (keyAction.actionType) {
-        case 'switchLayer':
-            serializeSwitchLayerAction(keyAction);
-            break;
-        default:
-            throw 'KeyAction doesn\'t have a valid actionType property: ' + keyAction.actionType;
-    }
-}
-
-function serializeSwitchLayerAction(switchLayerAction) {
-    writer.writeUInt8({
-        mod  : SWITCH_LAYER_MOD,
-        fn   : SWITCH_LAYER_FN,
-        mouse: SWITCH_LAYER_MOD
-    }[switchLayerAction] | switchLayerAction.toggle ? SWITCH_LAYER_TOGGLE : 0);
-}
-
 fs.writeFileSync('uhk-config.bin', writer.buffer);