Don't create duplicate tags; Tagging is idempotent

[vopamoi] / vopamoi.ts

diff --git a/vopamoi.ts b/vopamoi.ts
index 8112c4f009930309acf373a69e1581f57dc1e486..3b2f1638beb180e3e6b294cfa59b23ee2c73fe3d 100644 (file)
--- a/vopamoi.ts
+++ b/vopamoi.ts
@@ -41,6 +41,19 @@ const Model = {
     return task;
   },
 
+  addTag: function (createTimestamp: string, tagName: string): Element | null {
+    const task = this.getTask(createTimestamp);
+    if (!task) return null;
+    const existingTag = this.hasTag(task, tagName);
+    if (existingTag) return existingTag;
+    const tag = document.createElement("span");
+    tag.appendChild(document.createTextNode(tagName));
+    tag.classList.add("tag");
+    tag.setAttribute("tabindex", "0");
+    task.appendChild(tag);
+    return tag;
+  },
+
   edit: function (createTimestamp: string, newDescription: string): Element | null {
     const target = this.getTask(createTimestamp);
     if (!target) return null;
@@ -66,6 +79,15 @@ const Model = {
     return target;
   },
 
+  hasTag: function (task: Element, tag: string): Element | null {
+    for (const child of task.children) {
+      if (child.classList.contains("tag") && child.textContent === tag) {
+        return child;
+      }
+    }
+    return null;
+  },
+
   getPriority: function (task: Element): number {
     if (task.hasAttribute("data-priority")) {
       return parseFloat(task.getAttribute("data-priority")!);
@@ -123,6 +145,10 @@ function Log(prefix: string = "vp-") {
         const [createTimestamp, state] = splitN(data, " ", 1);
         return Model.setState(timestamp, createTimestamp, state);
       }
+      if (command == "Tag") {
+        const [createTimestamp, tag] = splitN(data, " ", 1);
+        return Model.addTag(createTimestamp, tag);
+      }
     },
 
     record: function (entry: string) {
@@ -156,6 +182,10 @@ function UI() {
       undoLog.push(`State ${now} deleted`);
       return <Element>log.recordAndApply(`${now} Create ${description}`);
     },
+    addTag: function (createTimestamp: string, tag: string) {
+      // TODO: undo
+      return log.recordAndApply(`${clock.now()} Tag ${createTimestamp} ${tag}`);
+    },
     edit: function (createTimestamp: string, newDescription: string, oldDescription: string) {
       undoLog.push(`Edit ${createTimestamp} ${oldDescription}`);
       return log.recordAndApply(`${clock.now()} Edit ${createTimestamp} ${newDescription}`);
@@ -212,6 +242,17 @@ function BrowserUI() {
       input.value = oldDescription;
       input.addEventListener("blur", this.completeEdit, { once: true });
       task.textContent = "";
+      task.insertBefore(input, task.firstChild);
+      input.focus();
+      event.preventDefault();
+    },
+
+    beginTagEdit: function (event: Event) {
+      const task = document.activeElement;
+      if (!task) return;
+      const input = document.createElement("input");
+      input.classList.add("tag");
+      input.addEventListener("blur", this.completeTagEdit, { once: true });
       task.appendChild(input);
       input.focus();
       event.preventDefault();
@@ -233,6 +274,18 @@ function BrowserUI() {
       }
     },
 
+    completeTagEdit: function (event: Event, resolution: CommitOrAbort = CommitOrAbort.Commit) {
+      const input = event.target as HTMLInputElement;
+      const task = input.parentElement!;
+      const newTagName = input.value;
+      input.removeEventListener("blur", this.completeTagEdit);
+      task.removeChild(input);
+      task.focus();
+      if (!Model.hasTag(task, newTagName)) {
+        ui.addTag(task.getAttribute("data-created")!, newTagName);
+      }
+    },
+
     firstVisibleTask: function () {
       for (const task of document.getElementsByClassName("task")) {
         if (task instanceof HTMLElement && task.getAttribute("data-state") === currentViewState) {
@@ -351,6 +404,9 @@ function handleKey(event: any) {
     if (event.target.id === "taskName") {
       if (event.key == "Enter") return browserUI.addTask(event);
       if (event.key == "Escape") return browserUI.returnFocusAfterInput();
+    } else if (event.target.classList.contains("tag")) {
+      if (event.key == "Enter") return browserUI.completeTagEdit(event);
+      if (event.key == "Escape") return browserUI.completeTagEdit(event, CommitOrAbort.Abort);
     } else {
       if (event.key == "Enter") return browserUI.completeEdit(event);
       if (event.key == "Escape") return browserUI.completeEdit(event, CommitOrAbort.Abort);
@@ -370,6 +426,7 @@ function handleKey(event: any) {
       if (event.key == "X") return browserUI.setState("deleted");
       if (event.key == "u") return browserUI.undo();
       if (event.key == "e") return browserUI.beginEdit(event);
+      if (event.key == "t") return browserUI.beginTagEdit(event);
       if (event.key == "v") return (inputState = InputState.View);
     } else if (inputState === InputState.View) {
       inputState = InputState.Command;