fix: apply new node value on blur (fixes #4)

rychkog · Jun 29, 2016 · 378a36f · 378a36f
1 parent 5732a92
commit 378a36f
Showing 1 changed file with 8 additions and 12 deletions.
diff --git a/src/editable/node-editable.directive.ts b/src/editable/node-editable.directive.ts
@@ -34,23 +34,19 @@ export class NodeEditableDirective implements OnInit {
 
   @HostListener('keyup.enter', ['$event.target.value'])
   private applyNewValue(newNodeValue: string) {
-    return this.valueChanged.emit({type: 'keyup', value: newNodeValue});
+    this.valueChanged.emit({type: 'keyup', value: newNodeValue});
   }
 
-  @HostListener('keyup.esc')
-  private cancelEditing() {
-    return this.valueChanged.emit({
-      type: 'keyup', 
-      value: this.nodeValue, 
-      action: NodeEditableEventAction.Cancel
-    });
+  @HostListener('blur', ['$event.target.value'])
+  private applyNewValueByLoosingFocus(newNodeValue: string): void {
+    this.valueChanged.emit({type: 'blur', value: newNodeValue});
   }
 
-  @HostListener('blur')
-  private cancelEditingByLoosingFocus() {
+  @HostListener('keyup.esc')
+  private cancelEditing(): void {
     this.valueChanged.emit({
-      type: 'blur', 
-      value: this.nodeValue, 
+      type: 'keyup',
+      value: this.nodeValue,
       action: NodeEditableEventAction.Cancel
     });
   }