moveCursor returns whether it was able to or not

[vopamoi] / vopamoi.ts

diff --git a/vopamoi.ts b/vopamoi.ts
index 4d6cd86bfe25289cca1a52969530f6e17851dfe2..bc7acb79604733e42bc570409f62fa6a54fd874d 100644 (file)
--- a/vopamoi.ts
+++ b/vopamoi.ts
@@ -14,18 +14,19 @@ function splitN(str: string, delimiter: string, limit: number = MAX_SAFE_INTEGER
 }
 
 const Model = {
-  createTask: function (description: string) {
+  createTask: function (timestamp: string, description: string) {
     const task = document.createElement("div");
     task.appendChild(document.createTextNode(description));
     task.setAttribute("tabindex", "0");
+    task.setAttribute("data-created", timestamp);
     return task;
   },
 
-  addTask: function (description: string) {
-    document.body.appendChild(this.createTask(description)).focus();
+  addTask: function (timestamp: string, description: string) {
+    document.body.appendChild(this.createTask(timestamp, description)).focus();
   },
 
-  moveCursor: function (offset: number) {
+  moveCursor: function (offset: number): boolean {
     var active = document.activeElement;
     if (offset === 1 && active) {
       active = active.nextElementSibling;
@@ -33,31 +34,31 @@ const Model = {
     if (offset === -1 && active) {
       active = active.previousElementSibling;
     }
-    if (active && active instanceof HTMLElement) active.focus();
+    if (active && active instanceof HTMLElement) {
+      active.focus();
+      return true;
+    }
+    return false;
   },
 };
 
 const Log = (function () {
   var next_log_index = 0;
   return {
-    addTask: function (description: string) {
-      this.recordAndApplyLogEntry(`${Date.now()} Create ${description}`);
-    },
-
-    applyLogEntry: function (entry: string) {
+    apply: function (entry: string) {
       const [timestamp, command, data] = splitN(entry, " ", 2);
       if (command == "Create") {
-        Model.addTask(data);
+        Model.addTask(timestamp, data);
       }
     },
 
-    recordLogEntry: function (entry: string) {
+    record: function (entry: string) {
       window.localStorage.setItem(`${next_log_index++}`, entry);
     },
 
-    recordAndApplyLogEntry: function (entry: string) {
-      this.recordLogEntry(entry);
-      this.applyLogEntry(entry);
+    recordAndApply: function (entry: string) {
+      this.record(entry);
+      this.apply(entry);
     },
 
     replay: function () {
@@ -66,27 +67,35 @@ const Log = (function () {
         if (entry === null) {
           break;
         }
-        this.applyLogEntry(entry);
+        this.apply(entry);
         next_log_index++;
       }
     },
   };
 })();
 
+const UI = {
+  addTask: function (description: string) {
+    Log.recordAndApply(`${Date.now()} Create ${description}`);
+  },
+};
+
+function focusTaskNameInput(event: any) {
+  document.getElementById("taskName")!.focus();
+  event.preventDefault();
+}
+
 function handleKey(event: any) {
   if (event.target.tagName !== "INPUT") {
-    if (event.key == "j") Model.moveCursor(1);
-    if (event.key == "k") Model.moveCursor(-1);
-    if (event.key == "c") {
-      document.getElementById("taskName")!.focus();
-      event.preventDefault();
-    }
+    if (event.key == "j") return Model.moveCursor(1);
+    if (event.key == "k") return Model.moveCursor(-1);
+    if (event.key == "c") return focusTaskNameInput(event);
   }
 }
 
 function browserCreateTask(form: any) {
   if (form.taskName.value) {
-    Log.addTask(form.taskName.value);
+    UI.addTask(form.taskName.value);
   }
   form.taskName.value = "";
   return false;