refactor save on blur

public/index.html

@@ -33,7 +33,6 @@
     <script src="scripts/AppFlip.js"></script>
     <script src="scripts/AppFps.js"></script>
     <script src="scripts/AppIcon.js"></script>
-    <script src="scripts/AppLateBlur.js"></script>
     <script src="scripts/AppSortable.js"></script>
     <script src="scripts/TodoApp.js"></script>
     <script src="scripts/TodoCustomList.js"></script>

public/scripts/AppDraggable.js

@@ -17,14 +17,8 @@ VT.AppDraggable = function (el, options) {
   var imageX, imageY;
   var currentTarget;
 
-  if (window.navigator.pointerEnabled) {
-    el.addEventListener('pointerdown', start);
-  } else if (window.navigator.msPointerEnabled) {
-    el.addEventListener('MSPointerDown', start);
-  } else {
-    el.addEventListener('mousedown', start);
-    el.addEventListener('touchstart', start);
-  }
+  el.addEventListener('touchstart', start);
+  el.addEventListener('mousedown', start);
 
   // maybe prevent click
   el.addEventListener(
@@ -90,7 +84,6 @@ VT.AppDraggable = function (el, options) {
 
   function end(e) {
     e.preventDefault();
-    e.stopImmediatePropagation();
 
     if (!dragging) {
       e.target.click();
@@ -110,33 +103,17 @@ VT.AppDraggable = function (el, options) {
   }
 
   function startListening() {
-    if (window.navigator.pointerEnabled) {
-      el.addEventListener('pointermove', move);
-      el.addEventListener('pointerup', end);
-    } else if (window.navigator.msPointerEnabled) {
-      el.addEventListener('MSPointerMove', move);
-      el.addEventListener('MSPointerUp', end);
-    } else {
-      window.addEventListener('mousemove', move);
-      window.addEventListener('mouseup', end);
-      el.addEventListener('touchmove', move);
-      el.addEventListener('touchend', end);
-    }
+    el.addEventListener('touchmove', move);
+    el.addEventListener('touchend', end);
+    window.addEventListener('mousemove', move);
+    window.addEventListener('mouseup', end);
   }
 
   function stopListening() {
-    if (window.navigator.pointerEnabled) {
-      el.removeEventListener('pointermove', move);
-      el.removeEventListener('pointerup', end);
-    } else if (window.navigator.msPointerEnabled) {
-      el.removeEventListener('MSPointerMove', move);
-      el.removeEventListener('MSPointerUp', end);
-    } else {
-      window.removeEventListener('mousemove', move);
-      window.removeEventListener('mouseup', end);
-      el.removeEventListener('touchmove', move);
-      el.removeEventListener('touchend', end);
-    }
+    el.removeEventListener('touchmove', move);
+    el.removeEventListener('touchend', end);
+    window.removeEventListener('mousemove', move);
+    window.removeEventListener('mouseup', end);
   }
 
   //

public/scripts/AppLateBlur.js

@@ -1,39 +0,0 @@
-/* global VT */
-window.VT = window.VT || {};
-
-/**
- * Enables `lateBlur` events on the target element.
- * After an actual `blur` event, subsequent interactions with the window
- * (e.g. focus, select, mouseup etc.) will dispatch a `lateBlur` event.
- */
-VT.AppLateBlur = function (el) {
-  el.addEventListener('blur', function () {
-    window.addEventListener('focus', dispatch);
-    window.addEventListener('select', dispatch);
-
-    if (window.navigator.pointerEnabled) {
-      window.addEventListener('pointerup', dispatch);
-    } else if (window.navigator.msPointerEnabled) {
-      window.addEventListener('MSPointerUp', dispatch);
-    } else {
-      window.addEventListener('mouseup', dispatch);
-      window.addEventListener('touchend', dispatch);
-    }
-  });
-
-  function dispatch() {
-    window.removeEventListener('focus', dispatch);
-    window.removeEventListener('select', dispatch);
-
-    if (window.navigator.pointerEnabled) {
-      window.removeEventListener('pointerup', dispatch);
-    } else if (window.navigator.msPointerEnabled) {
-      window.removeEventListener('MSPointerUp', dispatch);
-    } else {
-      window.removeEventListener('mouseup', dispatch);
-      window.removeEventListener('touchend', dispatch);
-    }
-
-    el.dispatchEvent(new CustomEvent('lateBlur', { bubbles: true }));
-  }
-};

public/scripts/TodoCustomList.js

@@ -6,7 +6,8 @@ VT.TodoCustomList = function (el) {
     list: null,
     editing: false,
   };
-  var focus = false;
+  var startEditing = false;
+  var saveOnBlur = true;
 
   el.innerHTML = [
     '<div class="header">',
@@ -26,31 +27,35 @@ VT.TodoCustomList = function (el) {
   VT.AppDraggable(titleEl, {
     dropSelector: '.todo-frame.-custom .container',
   });
-  VT.AppLateBlur(inputEl);
   VT.TodoList(el.querySelector('.todo-list'));
   el.querySelectorAll('.app-icon').forEach(VT.AppIcon);
 
   titleEl.addEventListener('click', function () {
-    focus = true;
+    startEditing = true;
     update({ editing: true });
   });
 
-  inputEl.addEventListener('input', function () {
-    el.dispatchEvent(
-      new CustomEvent('saveList', {
-        detail: { list: state.list, title: inputEl.value.trim() },
-        bubbles: true,
-      })
-    );
+  deleteEl.addEventListener('touchstart', function () {
+    saveOnBlur = false;
   });
 
-  inputEl.addEventListener('lateBlur', function () {
-    update({ editing: false });
+  deleteEl.addEventListener('mousedown', function () {
+    saveOnBlur = false;
   });
 
-  inputEl.addEventListener('keypress', function (e) {
-    if (e.keyCode === 13) {
-      update({ editing: false });
+  inputEl.addEventListener('blur', function () {
+    if (saveOnBlur) save();
+    saveOnBlur = true;
+  });
+
+  inputEl.addEventListener('keyup', function (e) {
+    switch (e.keyCode) {
+      case 13: // enter
+        save();
+        break;
+      case 27: // escape
+        cancelEdit();
+        break;
     }
   });
 
@@ -103,11 +108,26 @@ VT.TodoCustomList = function (el) {
     update: update,
   };
 
+  function save() {
+    el.dispatchEvent(
+      new CustomEvent('saveList', {
+        detail: { list: state.list, title: inputEl.value.trim() },
+        bubbles: true,
+      })
+    );
+    update({ editing: false });
+  }
+
+  function cancelEdit() {
+    saveOnBlur = false;
+    update({ editing: false });
+  }
+
   function update(next) {
     Object.assign(state, next);
 
     titleEl.innerText = state.list.title || '...';
-    inputEl.value = state.list.title;
+
     el.querySelector('.todo-list').todoList.update({ items: state.list.items });
     el.querySelector('.todo-list > .todo-item-input').setAttribute(
       'data-key',
@@ -116,10 +136,11 @@ VT.TodoCustomList = function (el) {
 
     el.classList.toggle('-editing', state.editing);
 
-    if (state.editing && focus) {
+    if (state.editing && startEditing) {
+      inputEl.value = state.list.title;
       inputEl.focus();
       inputEl.select();
-      focus = false;
+      startEditing = false;
     }
   }
 };

public/scripts/TodoItem.js

@@ -6,7 +6,8 @@ VT.TodoItem = function (el) {
     item: null,
     editing: false,
   };
-  var focus = false;
+  var startEditing = false;
+  var saveOnBlur = true;
 
   el.innerHTML = [
     '<div class="checkbox">',
@@ -19,17 +20,28 @@ VT.TodoItem = function (el) {
     '</p>',
   ].join('\n');
 
-  var inputEl = el.querySelector('.input');
+  var checkboxEl = el.querySelector('.checkbox');
   var labelEl = el.querySelector('.label');
+  var inputEl = el.querySelector('.input');
+  var saveEl = el.querySelector('.save');
 
   VT.AppDraggable(el, {
     dropSelector: '.todo-list > .items',
   });
-  VT.AppLateBlur(inputEl);
 
   el.querySelectorAll('.app-icon').forEach(VT.AppIcon);
 
-  el.querySelector('.checkbox').addEventListener('click', function () {
+  checkboxEl.addEventListener('touchstart', function () {
+    saveOnBlur = false;
+  });
+
+  checkboxEl.addEventListener('mousedown', function () {
+    saveOnBlur = false;
+  });
+
+  checkboxEl.addEventListener('click', function () {
+    if (state.editing) save();
+
     el.dispatchEvent(
       new CustomEvent('checkItem', {
         detail: {
@@ -42,23 +54,32 @@ VT.TodoItem = function (el) {
   });
 
   labelEl.addEventListener('click', function () {
-    focus = true;
+    startEditing = true;
     update({ editing: true });
   });
 
-  el.querySelector('.save').addEventListener('click', function () {
-    save();
+  inputEl.addEventListener('keyup', function (e) {
+    switch (e.keyCode) {
+      case 13: // enter
+        save();
+        break;
+      case 27: // escape
+        cancelEdit();
+        break;
+    }
   });
 
-  inputEl.addEventListener('keypress', function (e) {
-    if (e.keyCode === 13) save();
+  inputEl.addEventListener('blur', function () {
+    if (saveOnBlur) save();
+    saveOnBlur = true;
   });
 
-  inputEl.addEventListener('lateBlur', function () {
-    save();
-    update({ editing: false });
+  saveEl.addEventListener('mousedown', function () {
+    saveOnBlur = false;
   });
 
+  saveEl.addEventListener('click', save);
+
   el.addEventListener('draggableStart', function (e) {
     e.detail.data.item = state.item;
     e.detail.data.key = state.item.id;
@@ -95,21 +116,27 @@ VT.TodoItem = function (el) {
     update({ editing: false });
   }
 
+  function cancelEdit() {
+    saveOnBlur = false;
+    update({ editing: false });
+  }
+
   function update(next) {
     // TODO optimize
     Object.assign(state, next);
 
     el.classList.toggle('-done', state.item.done);
-    el.querySelector('.checkbox > input').checked = state.item.done;
+    checkboxEl.querySelector('input').checked = state.item.done;
     labelEl.innerText = state.item.label;
-    inputEl.value = state.item.label;
+
     el.classList.toggle('-editing', state.editing);
     el.classList.toggle('_nodrag', state.editing);
 
-    if (state.editing && focus) {
-      el.querySelector('.input').focus();
-      el.querySelector('.input').select();
-      focus = false;
+    if (state.editing && startEditing) {
+      inputEl.value = state.item.label;
+      inputEl.focus();
+      inputEl.select();
+      startEditing = false;
     }
   }
 };

public/scripts/TodoItemInput.js

@@ -2,6 +2,8 @@
 window.VT = window.VT || {};
 
 VT.TodoItemInput = function (el) {
+  var saveOnBlur = true;
+
   el.innerHTML = [
     '<input class="input" type="text">',
     '<button class="app-button save"><i class="app-icon" data-id="plus-24"></i></button>',
@@ -10,10 +12,9 @@ VT.TodoItemInput = function (el) {
   var inputEl = el.querySelector('.input');
   var saveEl = el.querySelector('.save');
 
-  VT.AppLateBlur(inputEl);
   el.querySelectorAll('.app-icon').forEach(VT.AppIcon);
 
-  inputEl.addEventListener('keypress', function (e) {
+  inputEl.addEventListener('keyup', function (e) {
     switch (e.keyCode) {
       case 13: // enter
         save();
@@ -24,7 +25,14 @@ VT.TodoItemInput = function (el) {
     }
   });
 
-  inputEl.addEventListener('lateBlur', save);
+  inputEl.addEventListener('blur', function () {
+    if (saveOnBlur) save();
+    saveOnBlur = true;
+  });
+
+  saveEl.addEventListener('mousedown', function () {
+    saveOnBlur = false;
+  });
 
   saveEl.addEventListener('click', function () {
     save();

public/styles/todo-item.css

@@ -53,6 +53,7 @@
   font-family: inherit;
   font-size: inherit;
   line-height: 1.5em;
+  background: transparent;
 }
 
 .todo-item > .form > .save {