Allow Model to return values through UI

[vopamoi] / vopamoi.ts

diff --git a/vopamoi.ts b/vopamoi.ts
index b899c7144d6acb5d6c8594b49c80805f1362e9ca..ba5ae85a16203adaa1e8615dc0b06ad0cdb35d9c 100644 (file)
--- a/vopamoi.ts
+++ b/vopamoi.ts
@@ -14,16 +14,14 @@ function splitN(str: string, delimiter: string, limit: number = MAX_SAFE_INTEGER
 }
 
 const Model = {
-  addTask: function (timestamp: string, description: string) {
-    document.getElementById("tasks")!.appendChild(this.createTask(timestamp, description)).focus();
-  },
-
-  createTask: function (timestamp: string, description: string) {
+  addTask: function (timestamp: string, description: string): Element {
     const task = document.createElement("div");
     task.appendChild(document.createTextNode(description));
     task.setAttribute("class", "task");
     task.setAttribute("tabindex", "0");
     task.setAttribute("data-created", timestamp);
+    document.getElementById("tasks")!.appendChild(task);
+    task.focus();
     return task;
   },
 
@@ -81,18 +79,18 @@ function Log(prefix: string = "vp-") {
     apply: function (entry: string) {
       const [timestamp, command, data] = splitN(entry, " ", 2);
       if (command == "Create") {
-        Model.addTask(timestamp, data);
+        return Model.addTask(timestamp, data);
       }
       if (command == "Destroy") {
-        Model.destroyTask(data.split(" ", 1)[0]);
+        return Model.destroyTask(data.split(" ", 1)[0]);
       }
       if (command == "State") {
         const [createTimestamp, state] = splitN(data, " ", 1);
-        Model.setState(timestamp, createTimestamp, state);
+        return Model.setState(timestamp, createTimestamp, state);
       }
       if (command == "Priority") {
         const [createTimestamp, newPriority] = splitN(data, " ", 1);
-        Model.setPriority(createTimestamp, parseFloat(newPriority));
+        return Model.setPriority(createTimestamp, parseFloat(newPriority));
       }
     },
 
@@ -102,7 +100,7 @@ function Log(prefix: string = "vp-") {
 
     recordAndApply: function (entry: string) {
       this.record(entry);
-      this.apply(entry);
+      return this.apply(entry);
     },
 
     replay: function () {
@@ -120,27 +118,27 @@ function Log(prefix: string = "vp-") {
 const log = Log();
 
 const UI = {
-  addTask: function (description: string) {
-    log.recordAndApply(`${Date.now()} Create ${description}`);
+  addTask: function (description: string): Element {
+    return <Element>log.recordAndApply(`${Date.now()} Create ${description}`);
   },
   destroyTask: function (createTimestamp: string) {
-    log.recordAndApply(`${Date.now()} Destroy ${createTimestamp} ${Model.getTask(createTimestamp)?.textContent}`);
+    return log.recordAndApply(`${Date.now()} Destroy ${createTimestamp} ${Model.getTask(createTimestamp)?.textContent}`);
   },
   setPriority: function (createTimestamp: string, priority: number) {
-    log.recordAndApply(`${Date.now()} Priority ${createTimestamp} ${priority}`);
+    return log.recordAndApply(`${Date.now()} Priority ${createTimestamp} ${priority}`);
   },
   setState: function (createTimestamp: string, state: string) {
-    log.recordAndApply(`${Date.now()} State ${createTimestamp} ${state}`);
+    return log.recordAndApply(`${Date.now()} State ${createTimestamp} ${state}`);
   },
 };
 
 const BrowserUI = {
-  addTask: function (form: any) {
-    if (form.taskName.value) {
-      UI.addTask(form.taskName.value);
-      form.taskName.value = "";
+  addTask: function () {
+    const input = <HTMLInputElement>document.getElementById("taskName");
+    if (input.value) {
+      const task = UI.addTask(input.value);
+      input.value = "";
     }
-    return false;
   },
 
   destroyTask: function () {
@@ -216,7 +214,9 @@ const BrowserUI = {
 };
 
 function handleKey(event: any) {
-  if (event.target.tagName !== "INPUT") {
+  if (event.target.tagName === "INPUT") {
+    if (event.key == "Enter") return BrowserUI.addTask();
+  } else {
     if (event.key == "j") return BrowserUI.moveCursor(1);
     if (event.key == "k") return BrowserUI.moveCursor(-1);
     if (event.key == "J") return BrowserUI.moveTask(1);