Merge pull request #11 from lifenautjoe/bugfix/enter-should-remove-focus

Bugfix/enter should remove focus
lifenautjoe · Apr 8, 2018 · c288bb8 · c288bb8
2 parents 00b8724 + 1a5bfb5
commit c288bb8
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/src/droppable.ts b/src/droppable.ts
@@ -160,6 +160,7 @@ export default class Droppable {
     private onElementKeyDown(e: { [key: string]: any }) {
         if (e['keyCode'] === Droppable.ENTER_KEY_CODE) {
             this.promptForFiles();
+            this.element.blur();
         }
     }
 

diff --git a/test/droppable.test.ts b/test/droppable.test.ts
@@ -488,6 +488,22 @@ describe('Droppable', () => {
 
                     expect(mockFn).toHaveBeenCalled();
                 });
+
+                it('should call this.element.blur()', () => {
+                    const element = document.createElement('div');
+                    const droppable = new Droppable({
+                        element
+                    });
+
+                    const mockFn = spyOn(element, 'blur');
+
+                    droppable['promptForFiles'] = () => {};
+
+                    const keyDownEvent = new KeyboardEvent('keydown', { keyCode: 13 });
+                    droppable['onElementKeyDown'](keyDownEvent);
+
+                    expect(mockFn).toHaveBeenCalled();
+                });
             });
 
             describe('when the key is not enter', () => {