Always set state attribute

diff --git a/vopamoi.ts b/vopamoi.ts
index 4280ffcf9fde5c858cc48e1794aacd2d00719524..b16bc703b2ee1ca8432c0f3c2bab782fed502421 100644 (file)
--- a/vopamoi.ts
+++ b/vopamoi.ts
@@ -36,6 +36,7 @@ const Model = {
     task.setAttribute("class", "task");
     task.setAttribute("tabindex", "0");
     task.setAttribute("data-created", timestamp);
     task.setAttribute("class", "task");
     task.setAttribute("tabindex", "0");
     task.setAttribute("data-created", timestamp);

+    task.setAttribute("data-state", "todo");

     document.getElementById("tasks")!.appendChild(task);
     return task;
   },
     document.getElementById("tasks")!.appendChild(task);
     return task;
   },


@@ -73,10 +74,6 @@ const Model = {
     return parseFloat(task.getAttribute("data-created")!);
   },
 
     return parseFloat(task.getAttribute("data-created")!);
   },
 

-  getState: function (task: Element): string {
-    return task.getAttribute("data-state") ?? "todo";
-  },
-

   getTask: function (createTimestamp: string) {
     for (const task of document.getElementsByClassName("task")) {
       if (task.getAttribute("data-created") === createTimestamp) {
   getTask: function (createTimestamp: string) {
     for (const task of document.getElementsByClassName("task")) {
       if (task.getAttribute("data-created") === createTimestamp) {


@@ -297,7 +294,7 @@ const BrowserUI = {
   setState: function (newState: string) {
     const task = document.activeElement;
     if (!task) return;
   setState: function (newState: string) {
     const task = document.activeElement;
     if (!task) return;

-    const oldState = Model.getState(task);
+    const oldState = task.getAttribute("data-state");

     if (newState === oldState) return;
     const createTimestamp = task.getAttribute("data-created")!;
     this.moveCursor(1) || this.moveCursor(-1);
     if (newState === oldState) return;
     const createTimestamp = task.getAttribute("data-created")!;
     this.moveCursor(1) || this.moveCursor(-1);