Eliminate createTask. It's part of addTask

In the spirit of maintaining the invariant: all tasks are in the DOM; no
loose, floating tasks.

diff --git a/vopamoi.ts b/vopamoi.ts
index b899c7144d6acb5d6c8594b49c80805f1362e9ca..ab0615593b321d90b574a3d04539a99d45a3a700 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) {
 
 const Model = {
   addTask: function (timestamp: string, description: string) {

-    document.getElementById("tasks")!.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);
     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) {
   },
 
   destroyTask: function (createTimestamp: string) {