refactor

Signed-off-by: Jari Kolehmainen <jari.kolehmainen@gmail.com>

src/common/hotbar-store.ts

@@ -106,6 +106,26 @@ export class HotbarStore extends BaseStore<HotbarStoreModel> {
     }
   }
 
+  switchToPrevious() {
+    let index = hotbarStore.activeHotbarIndex - 1;
+
+    if (index < 0) {
+      index = hotbarStore.hotbars.length - 1;
+    }
+
+    hotbarStore.activeHotbarId = hotbarStore.hotbars[index].id;
+  }
+
+  switchToNext() {
+    let index = hotbarStore.activeHotbarIndex + 1;
+
+    if (index >= hotbarStore.hotbars.length) {
+      index = 0;
+    }
+
+    hotbarStore.activeHotbarId = hotbarStore.hotbars[index].id;
+  }
+
   toJSON(): HotbarStoreModel {
     const model: HotbarStoreModel = {
       hotbars: this.hotbars,

src/renderer/components/hotbar/hotbar-menu.tsx

@@ -44,23 +44,11 @@ export class HotbarMenu extends React.Component<Props> {
   }
 
   previous() {
-    let index = hotbarStore.activeHotbarIndex - 1;
-
-    if (index < 0) {
-      index = hotbarStore.hotbars.length - 1;
-    }
-
-    hotbarStore.activeHotbarId = hotbarStore.hotbars[index].id;
+    hotbarStore.switchToPrevious();
   }
 
   next() {
-    let index = hotbarStore.activeHotbarIndex + 1;
-
-    if (index >= hotbarStore.hotbars.length) {
-      index = 0;
-    }
-
-    hotbarStore.activeHotbarId = hotbarStore.hotbars[index].id;
+    hotbarStore.switchToNext();
   }
 
   openSelector() {