Fix: Allow textarea to be interactable in certain scenarios (#174)

box · Apr 16, 2018 · 9ce5b0a · 9ce5b0a
1 parent aea1cfa
commit 9ce5b0a
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/controllers/PointModeController.js b/src/controllers/PointModeController.js
@@ -115,7 +115,7 @@ class PointModeController extends AnnotationModeController {
         this.pointClickHandler = this.pointClickHandler.bind(this);
 
         // Get handlers
-        this.pushElementHandler(this.annotatedElement, ['mousedown', 'touchstart'], this.pointClickHandler);
+        this.pushElementHandler(this.annotatedElement, ['click', 'touchstart'], this.pointClickHandler);
 
         this.pushElementHandler(this.exitButtonEl, 'click', this.toggleMode);
     }

diff --git a/src/controllers/__tests__/PointModeController-test.js b/src/controllers/__tests__/PointModeController-test.js
@@ -129,10 +129,20 @@ describe('controllers/PointModeController', () => {
 
     describe('setupHandlers()', () => {
         it('should successfully contain mode handlers', () => {
+            sandbox.stub(controller, 'pushElementHandler');
             controller.exitButtonEl = 'also definitely not undefined';
 
             controller.setupHandlers();
-            expect(controller.handlers.length).to.equal(2);
+            expect(controller.pushElementHandler).to.be.calledWith(
+                controller.annotatedElement,
+                ['click', 'touchstart'],
+                controller.pointClickHandler
+            );
+            expect(controller.pushElementHandler).to.be.calledWith(
+                controller.exitButtonEl,
+                'click',
+                controller.toggleMode
+            );
         });
     });