Initiate task creation with Enter key rather than form submit

[vopamoi] / vopamoi.ts

diff --git a/vopamoi.ts b/vopamoi.ts
index ad1051ad5f9ffc6a4712dc1acada82944e643fac..fe3d491630f9e5781bdc7ba62f2f6ee0ad731b7e 100644 (file)
--- a/vopamoi.ts
+++ b/vopamoi.ts
@@ -15,16 +15,13 @@ function splitN(str: string, delimiter: string, limit: number = MAX_SAFE_INTEGER
 
 const Model = {
   addTask: function (timestamp: string, description: string) {
-    document.body.appendChild(this.createTask(timestamp, description)).focus();
-  },
-
-  createTask: function (timestamp: string, description: string) {
     const task = document.createElement("div");
     task.appendChild(document.createTextNode(description));
     task.setAttribute("class", "task");
     task.setAttribute("tabindex", "0");
     task.setAttribute("data-created", timestamp);
-    return task;
+    document.getElementById("tasks")!.appendChild(task);
+    task.focus();
   },
 
   destroyTask: function (createTimestamp: string) {
@@ -34,6 +31,13 @@ const Model = {
     }
   },
 
+  getPriority: function (task: Element): number {
+    if (task.hasAttribute("data-priority")) {
+      return parseFloat(task.getAttribute("data-priority")!);
+    }
+    return parseFloat(task.getAttribute("data-created")!);
+  },
+
   getTask: function (createTimestamp: string) {
     for (const task of document.getElementsByClassName("task")) {
       if (task.getAttribute("data-created") === createTimestamp) {
@@ -42,6 +46,21 @@ const Model = {
     }
   },
 
+  setPriority: function (createTimestamp: string, priority: number) {
+    const target = this.getTask(createTimestamp);
+    if (!target) return;
+    target.setAttribute("data-priority", `${priority}`);
+    for (const task of document.getElementsByClassName("task")) {
+      if (task !== target && this.getPriority(task) > priority) {
+        task.parentElement!.insertBefore(target, task);
+        target instanceof HTMLElement && target.focus();
+        return;
+      }
+    }
+    document.getElementById("tasks")!.appendChild(target);
+    target instanceof HTMLElement && target.focus();
+  },
+
   setState: function (stateTimestamp: string, createTimestamp: string, state: string) {
     const task = this.getTask(createTimestamp);
     if (task) {
@@ -68,6 +87,10 @@ function Log(prefix: string = "vp-") {
         const [createTimestamp, state] = splitN(data, " ", 1);
         Model.setState(timestamp, createTimestamp, state);
       }
+      if (command == "Priority") {
+        const [createTimestamp, newPriority] = splitN(data, " ", 1);
+        Model.setPriority(createTimestamp, parseFloat(newPriority));
+      }
     },
 
     record: function (entry: string) {
@@ -100,18 +123,21 @@ const UI = {
   destroyTask: function (createTimestamp: string) {
     log.recordAndApply(`${Date.now()} Destroy ${createTimestamp} ${Model.getTask(createTimestamp)?.textContent}`);
   },
+  setPriority: function (createTimestamp: string, priority: number) {
+    log.recordAndApply(`${Date.now()} Priority ${createTimestamp} ${priority}`);
+  },
   setState: function (createTimestamp: string, state: string) {
     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) {
+      UI.addTask(input.value);
+      input.value = "";
     }
-    return false;
   },
 
   destroyTask: function () {
@@ -152,6 +178,33 @@ const BrowserUI = {
     return false;
   },
 
+  moveTask: function (offset: number) {
+    const active = document.activeElement;
+    if (!active) return;
+    const dest = this.visibleTaskAtOffset(active, offset);
+    if (dest === active) return; // Already extremal
+    var onePastDest: Element | null = this.visibleTaskAtOffset(dest, offset / Math.abs(offset));
+    if (onePastDest == dest) onePastDest = null; // Will become extremal
+    if (offset > 0) {
+      this.setPriority(active, dest, onePastDest);
+    } else {
+      this.setPriority(active, onePastDest, dest);
+    }
+  },
+
+  // Change task's priority to be between other tasks a and b.
+  setPriority: function (task: Element, a: Element | null, b: Element | null) {
+    const aPriority = a === null ? 0 : Model.getPriority(a);
+    const bPriority = b === null ? Date.now() : Model.getPriority(b);
+    console.assert(aPriority < bPriority, aPriority, "<", bPriority);
+    const span = bPriority - aPriority;
+    const newPriority = aPriority + 0.1 * span + 0.8 * span * Math.random();
+    console.assert(aPriority < newPriority && newPriority < bPriority, aPriority, "<", newPriority, "<", bPriority);
+    const newPriorityRounded = Math.round(newPriority);
+    const okToRound = aPriority < newPriorityRounded && newPriorityRounded < bPriority;
+    UI.setPriority(task.getAttribute("data-created")!, okToRound ? newPriorityRounded : newPriority);
+  },
+
   setState: function (state: string) {
     const createTimestamp = document.activeElement?.getAttribute("data-created");
     this.moveCursor(1) || this.moveCursor(-1);
@@ -160,9 +213,13 @@ 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);
+    if (event.key == "K") return BrowserUI.moveTask(-1);
     if (event.key == "n") return BrowserUI.focusTaskNameInput(event);
     if (event.key == "s") return BrowserUI.setState("someday-maybe");
     if (event.key == "w") return BrowserUI.setState("waiting");