Factor out task offset logic

[vopamoi] / vopamoi.ts

diff --git a/vopamoi.ts b/vopamoi.ts
index 2b38dd88b35c2f88e47b2f1d54d33c4a9d2c3c02..3042adde1255c00fece833224eb7af7f3521c990 100644 (file)
--- a/vopamoi.ts
+++ b/vopamoi.ts
@@ -53,7 +53,7 @@ const Model = {
   },
 };
 
-const Log = (function () {
+function Log(prefix: string = "vp-") {
   var next_log_index = 0;
   return {
     apply: function (entry: string) {
@@ -71,7 +71,7 @@ const Log = (function () {
     },
 
     record: function (entry: string) {
-      window.localStorage.setItem(`${next_log_index++}`, entry);
+      window.localStorage.setItem(`${prefix}${next_log_index++}`, entry);
     },
 
     recordAndApply: function (entry: string) {
@@ -81,7 +81,7 @@ const Log = (function () {
 
     replay: function () {
       while (true) {
-        const entry = window.localStorage.getItem(`${next_log_index}`);
+        const entry = window.localStorage.getItem(`${prefix}${next_log_index}`);
         if (entry === null) {
           break;
         }
@@ -90,17 +90,18 @@ const Log = (function () {
       }
     },
   };
-})();
+}
+const log = Log();
 
 const UI = {
   addTask: function (description: string) {
-    Log.recordAndApply(`${Date.now()} Create ${description}`);
+    log.recordAndApply(`${Date.now()} Create ${description}`);
   },
   destroyTask: function (createTimestamp: string) {
-    Log.recordAndApply(`${Date.now()} Destroy ${createTimestamp} ${Model.getTask(createTimestamp)?.textContent}`);
+    log.recordAndApply(`${Date.now()} Destroy ${createTimestamp} ${Model.getTask(createTimestamp)?.textContent}`);
   },
   setState: function (createTimestamp: string, state: string) {
-    Log.recordAndApply(`${Date.now()} State ${createTimestamp} ${state}`);
+    log.recordAndApply(`${Date.now()} State ${createTimestamp} ${state}`);
   },
 };
 
@@ -124,16 +125,28 @@ const BrowserUI = {
     event.preventDefault();
   },
 
-  moveCursor: function (offset: number): boolean {
-    var active = document.activeElement;
-    if (offset === 1 && active) {
-      active = active.nextElementSibling;
-    }
-    if (offset === -1 && active) {
-      active = active.previousElementSibling;
+  visibleTaskAtOffset(task: Element, offset: number): Element {
+    var cursor: Element | null = task;
+    var valid_cursor = cursor;
+    const increment = offset / Math.abs(offset);
+    while (true) {
+      cursor = increment > 0 ? cursor.nextElementSibling : cursor.previousElementSibling;
+      if (!cursor || !(cursor instanceof HTMLElement)) break;
+      if (cursor.style.display !== "none") {
+        offset -= increment;
+        valid_cursor = cursor;
+      }
+      if (Math.abs(offset) < 0.5) break;
     }
-    if (active && active instanceof HTMLElement) {
-      active.focus();
+    return valid_cursor;
+  },
+
+  moveCursor: function (offset: number): boolean {
+    const active = document.activeElement;
+    if (!active) return false;
+    const dest = this.visibleTaskAtOffset(active, offset);
+    if (dest !== active && dest instanceof HTMLElement) {
+      dest.focus();
       return true;
     }
     return false;
@@ -161,5 +174,5 @@ function handleKey(event: any) {
 
 function browserInit() {
   document.body.addEventListener("keydown", handleKey, { capture: false });
-  Log.replay();
+  log.replay();
 }