Don't attempt to create duplicate tags

[vopamoi] / vopamoi.ts

diff --git a/vopamoi.ts b/vopamoi.ts
index f93c51c45a134826c034626377e62457dfbfc3bb..7c51935a2c14535444199ddb68da35ec2eca4e17 100644 (file)
--- a/vopamoi.ts
+++ b/vopamoi.ts
@@ -33,7 +33,7 @@ const Model = {
   addTask: function (timestamp: string, description: string): Element {
     const task = document.createElement("div");
     task.appendChild(document.createTextNode(description));
   addTask: function (timestamp: string, description: string): Element {
     const task = document.createElement("div");
     task.appendChild(document.createTextNode(description));

-    task.setAttribute("class", "task");
+    task.classList.add("task");

     task.setAttribute("tabindex", "0");
     task.setAttribute("data-created", timestamp);
     task.setAttribute("data-state", "todo");
     task.setAttribute("tabindex", "0");
     task.setAttribute("data-created", timestamp);
     task.setAttribute("data-state", "todo");


@@ -41,20 +41,30 @@ const Model = {
     return task;
   },
 
     return task;
   },
 

+  addTag: function (createTimestamp: string, tagName: string): Element | null {
+    const task = this.getTask(createTimestamp);
+    if (!task) return null;
+    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;
     if (target.hasAttribute("data-description")) {
       // Oh no: An edit has arrived from a replica while a local edit is in progress.
   edit: function (createTimestamp: string, newDescription: string): Element | null {
     const target = this.getTask(createTimestamp);
     if (!target) return null;
     if (target.hasAttribute("data-description")) {
       // Oh no: An edit has arrived from a replica while a local edit is in progress.

-      const input = target.children[0] as HTMLInputElement;
+      const input = target.firstChild as HTMLInputElement;

       if (
         input.value === target.getAttribute("data-description") &&
       if (
         input.value === target.getAttribute("data-description") &&

-        input.selectionStart === 0 &&
+        input.selectionStart === input.value.length &&

         input.selectionEnd === input.value.length
       ) {
         // No local changes have actually been made yet.  Change the contents of the edit box!
         input.value = newDescription;
         input.selectionEnd === input.value.length
       ) {
         // No local changes have actually been made yet.  Change the contents of the edit box!
         input.value = newDescription;

-        input.select();

       } else {
         // No great options.
         // Prefer not to interrupt the local user's edit.
       } else {
         // No great options.
         // Prefer not to interrupt the local user's edit.


@@ -67,6 +77,15 @@ const Model = {
     return target;
   },
 
     return target;
   },
 

+  hasTag: function (task: Element, tag: string) {
+    for (const child of task.children) {
+      if (child.classList.contains("tag") && child.textContent === tag) {
+        return true;
+      }
+    }
+    return false;
+  },
+

   getPriority: function (task: Element): number {
     if (task.hasAttribute("data-priority")) {
       return parseFloat(task.getAttribute("data-priority")!);
   getPriority: function (task: Element): number {
     if (task.hasAttribute("data-priority")) {
       return parseFloat(task.getAttribute("data-priority")!);


@@ -116,13 +135,17 @@ function Log(prefix: string = "vp-") {
         const [createTimestamp, description] = splitN(data, " ", 1);
         return Model.edit(createTimestamp, description);
       }
         const [createTimestamp, description] = splitN(data, " ", 1);
         return Model.edit(createTimestamp, description);
       }

+      if (command == "Priority") {
+        const [createTimestamp, newPriority] = splitN(data, " ", 1);
+        return Model.setPriority(createTimestamp, parseFloat(newPriority));
+      }

       if (command == "State") {
         const [createTimestamp, state] = splitN(data, " ", 1);
         return Model.setState(timestamp, createTimestamp, state);
       }
       if (command == "State") {
         const [createTimestamp, state] = splitN(data, " ", 1);
         return Model.setState(timestamp, createTimestamp, state);
       }

-      if (command == "Priority") {
-        const [createTimestamp, newPriority] = splitN(data, " ", 1);
-        return Model.setPriority(createTimestamp, parseFloat(newPriority));
+      if (command == "Tag") {
+        const [createTimestamp, tag] = splitN(data, " ", 1);
+        return Model.addTag(createTimestamp, tag);

       }
     },
 
       }
     },
 


@@ -149,160 +172,224 @@ function Log(prefix: string = "vp-") {
 }
 const log = Log();
 
 }
 const log = Log();
 

-const undoLog: string[] = [];
+function UI() {
+  const undoLog: string[] = [];
+  return {
+    addTask: function (description: string): Element {
+      const now = clock.now();
+      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}`);
+    },
+    setPriority: function (createTimestamp: string, newPriority: number, oldPriority: number) {
+      undoLog.push(`Priority ${createTimestamp} ${oldPriority}`);
+      return log.recordAndApply(`${clock.now()} Priority ${createTimestamp} ${newPriority}`);
+    },
+    setState: function (createTimestamp: string, newState: string, oldState: string) {
+      undoLog.push(`State ${createTimestamp} ${oldState}`);
+      return log.recordAndApply(`${clock.now()} State ${createTimestamp} ${newState}`);
+    },
+    undo: function () {
+      if (undoLog.length > 0) {
+        return log.recordAndApply(`${clock.now()} ${undoLog.pop()}`);
+      }
+    },
+  };
+}
+const ui = UI();

 
 

-const UI = {
-  addTask: function (description: string): Element {
-    const now = clock.now();
-    undoLog.push(`State ${now} deleted`);
-    return <Element>log.recordAndApply(`${now} Create ${description}`);
-  },
-  edit: function (createTimestamp: string, newDescription: string, oldDescription: string) {
-    undoLog.push(`Edit ${createTimestamp} ${oldDescription}`);
-    return log.recordAndApply(`${clock.now()} Edit ${createTimestamp} ${newDescription}`);
-  },
-  setPriority: function (createTimestamp: string, newPriority: number, oldPriority: number) {
-    undoLog.push(`Priority ${createTimestamp} ${oldPriority}`);
-    return log.recordAndApply(`${clock.now()} Priority ${createTimestamp} ${newPriority}`);
-  },
-  setState: function (createTimestamp: string, newState: string, oldState: string) {
-    undoLog.push(`State ${createTimestamp} ${oldState}`);
-    return log.recordAndApply(`${clock.now()} State ${createTimestamp} ${newState}`);
-  },
-  undo: function () {
-    if (undoLog.length > 0) {
-      return log.recordAndApply(`${clock.now()} ${undoLog.pop()}`);
-    }
-  },
-};
+enum CommitOrAbort {
+  Commit,
+  Abort,
+}

 
 

-const BrowserUI = {
-  addTask: function (event: KeyboardEvent) {
-    const input = <HTMLInputElement>document.getElementById("taskName");
-    if (input.value) {
-      const task = UI.addTask(input.value);
-      if (task && task instanceof HTMLElement) task.focus();
-      input.value = "";
-      if (event.getModifierState("Control")) {
-        this.setPriority(task, null, document.getElementsByClassName("task")[0]);
+function BrowserUI() {
+  var currentViewState = "todo";
+  var taskFocusedBeforeJumpingToInput: HTMLElement | null = null;
+  return {
+    addTask: function (event: KeyboardEvent) {
+      const input = <HTMLInputElement>document.getElementById("taskName");
+      if (input.value) {
+        const task = ui.addTask(input.value);
+        if (currentViewState === "todo") {
+          task instanceof HTMLElement && task.focus();
+        } else if (this.returnFocusAfterInput()) {
+        } else {
+          this.firstVisibleTask()?.focus();
+        }
+        input.value = "";
+        if (event.getModifierState("Control")) {
+          this.setPriority(task, null, document.getElementsByClassName("task")[0]);
+        }

       }
       }

-    }
-  },
+    },

 
 

-  beginEdit: function (event: Event) {
-    const task = document.activeElement;
-    if (!task) return;
-    const input = document.createElement("input");
-    const oldDescription = task.textContent!;
-    task.setAttribute("data-description", oldDescription);
-    input.value = oldDescription;
-    input.addEventListener("blur", BrowserUI.completeEdit, { once: true });
-    task.textContent = "";
-    task.appendChild(input);
-    input.focus();
-    input.select();
-    event.preventDefault();
-  },
+    beginEdit: function (event: Event) {
+      const task = document.activeElement;
+      if (!task) return;
+      const input = document.createElement("input");
+      const oldDescription = task.textContent!;
+      task.setAttribute("data-description", oldDescription);
+      input.value = oldDescription;
+      input.addEventListener("blur", this.completeEdit, { once: true });
+      task.textContent = "";
+      task.insertBefore(input, task.firstChild);
+      input.focus();
+      event.preventDefault();
+    },

 
 

-  completeEdit: function (event: Event) {
-    const input = event.target as HTMLInputElement;
-    const task = input.parentElement!;
-    const oldDescription = task.getAttribute("data-description")!;
-    const newDescription = input.value;
-    input.removeEventListener("blur", BrowserUI.completeEdit);
-    task.removeChild(task.children[0]);
-    task.removeAttribute("data-description");
-    task.focus();
-    if (newDescription === oldDescription) {
-      task.textContent = oldDescription;
-    } else {
-      UI.edit(task.getAttribute("data-created")!, newDescription, oldDescription);
-    }
-  },
+    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();
+    },

 
 

-  firstVisibleTask: function () {
-    for (const task of document.getElementsByClassName("task")) {
-      if (task instanceof HTMLElement && task.getAttribute("data-state")! === "todo") {
-        return task;
+    completeEdit: function (event: Event, resolution: CommitOrAbort = CommitOrAbort.Commit) {
+      const input = event.target as HTMLInputElement;
+      const task = input.parentElement!;
+      const oldDescription = task.getAttribute("data-description")!;
+      const newDescription = input.value;
+      input.removeEventListener("blur", this.completeEdit);
+      task.removeChild(input);
+      task.removeAttribute("data-description");
+      task.focus();
+      if (newDescription === oldDescription || resolution === CommitOrAbort.Abort) {
+        task.textContent = oldDescription;
+      } else {
+        ui.edit(task.getAttribute("data-created")!, newDescription, oldDescription);

       }
       }

-    }
-  },
+    },

 
 

-  focusTaskNameInput: function (event: Event) {
-    document.getElementById("taskName")!.focus();
-    event.preventDefault();
-  },
+    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);
+      }
+    },

 
 

-  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.getAttribute("data-state")! === "todo") {
-        offset -= increment;
-        valid_cursor = cursor;
+    firstVisibleTask: function () {
+      for (const task of document.getElementsByClassName("task")) {
+        if (task instanceof HTMLElement && task.getAttribute("data-state") === currentViewState) {
+          return task;
+        }

       }
       }

-      if (Math.abs(offset) < 0.5) break;
-    }
-    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;
-  },
+    focusTaskNameInput: function (event: Event) {
+      if (document.activeElement instanceof HTMLElement) {
+        taskFocusedBeforeJumpingToInput = document.activeElement;
+      }
+      document.getElementById("taskName")!.focus();
+      event.preventDefault();
+    },

 
 

-  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);
-    }
-  },
+    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.getAttribute("data-state")! === currentViewState) {
+          offset -= increment;
+          valid_cursor = cursor;
+        }
+        if (Math.abs(offset) < 0.5) break;
+      }
+      return valid_cursor;
+    },

 
 

-  // 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 ? clock.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, Model.getPriority(task));
-    task instanceof HTMLElement && task.focus();
-  },
+    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;
+    },

 
 

-  setState: function (newState: string) {
-    const task = document.activeElement;
-    if (!task) return;
-    const oldState = task.getAttribute("data-state")!;
-    if (newState === oldState) return;
-    const createTimestamp = task.getAttribute("data-created")!;
-    this.moveCursor(1) || this.moveCursor(-1);
-    return UI.setState(createTimestamp, newState, oldState);
-  },
+    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);
+      }
+    },

 
 

-  undo: function () {
-    const ret = UI.undo();
-    if (ret && ret instanceof HTMLElement) ret.focus();
-  },
-};
+    returnFocusAfterInput: function (): boolean {
+      if (taskFocusedBeforeJumpingToInput) {
+        taskFocusedBeforeJumpingToInput.focus();
+        return true;
+      }
+      return false;
+    },
+
+    // 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 ? clock.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, Model.getPriority(task));
+      task instanceof HTMLElement && task.focus();
+    },
+
+    setState: function (newState: string) {
+      const task = document.activeElement;
+      if (!task) return;
+      const oldState = task.getAttribute("data-state")!;
+      if (newState === oldState) return;
+      const createTimestamp = task.getAttribute("data-created")!;
+      this.moveCursor(1) || this.moveCursor(-1);
+      return ui.setState(createTimestamp, newState, oldState);
+    },
+
+    setView: function (state: string) {
+      const sheet = (document.getElementById("viewStyle") as HTMLStyleElement).sheet!;
+      sheet.insertRule(`.task:not([data-state=${state}]) { display: none }`);
+      sheet.removeRule(1);
+      currentViewState = state;
+      if (document.activeElement?.getAttribute("data-state") !== state) {
+        this.firstVisibleTask()?.focus();
+      }
+    },
+
+    undo: function () {
+      const ret = ui.undo();
+      if (ret && ret instanceof HTMLElement) ret.focus();
+    },
+  };
+}
+const browserUI = BrowserUI();

 
 enum InputState {
   Command,
 
 enum InputState {
   Command,


@@ -313,28 +400,40 @@ var inputState = InputState.Command;
 function handleKey(event: any) {
   if (event.target.tagName === "INPUT") {
     if (event.target.id === "taskName") {
 function handleKey(event: any) {
   if (event.target.tagName === "INPUT") {
     if (event.target.id === "taskName") {

-      if (event.key == "Enter") return BrowserUI.addTask(event);
+      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 {
     } else {

-      if (event.key == "Enter") return BrowserUI.completeEdit(event);
+      if (event.key == "Enter") return browserUI.completeEdit(event);
+      if (event.key == "Escape") return browserUI.completeEdit(event, CommitOrAbort.Abort);

     }
   } else {
     if (inputState === InputState.Command) {
     }
   } else {
     if (inputState === InputState.Command) {

-      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");
-      if (event.key == "d") return BrowserUI.setState("done");
-      if (event.key == "c") return BrowserUI.setState("cancelled");
-      if (event.key == "t") return BrowserUI.setState("todo");
-      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 == "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 == "c") return browserUI.setState("cancelled");
+      if (event.key == "d") return browserUI.setState("done");
+      if (event.key == "q") return browserUI.setState("todo");
+      if (event.key == "s") return browserUI.setState("someday-maybe");
+      if (event.key == "w") return browserUI.setState("waiting");
+      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) {
       if (event.key == "v") return (inputState = InputState.View);
     } else if (inputState === InputState.View) {

-      return (inputState = InputState.Command);
+      inputState = InputState.Command;
+      if (event.key == "c") return browserUI.setView("cancelled");
+      if (event.key == "d") return browserUI.setView("done");
+      if (event.key == "q") return browserUI.setView("todo");
+      if (event.key == "s") return browserUI.setView("someday-maybe");
+      if (event.key == "w") return browserUI.setView("waiting");
+      if (event.key == "x") return browserUI.setView("deleted");

     }
   }
 }
     }
   }
 }


@@ -342,5 +441,5 @@ function handleKey(event: any) {
 function browserInit() {
   document.body.addEventListener("keydown", handleKey, { capture: false });
   log.replay();
 function browserInit() {
   document.body.addEventListener("keydown", handleKey, { capture: false });
   log.replay();

-  BrowserUI.firstVisibleTask()?.focus();
+  browserUI.firstVisibleTask()?.focus();

 }
 }