Fixed eraser and better naming

Author: Victor Nazzaro

src/UI/eraser.js

@@ -9,17 +9,24 @@ import {
 let _canerase = false;
 let userId = "user";
 
-function handleDocumentMouseDown(e){
+function handleDocumentDown(e){
   _canerase = true;
 }
 
-function handleDocumentMouseUp(e){
+function handleDocumentUp(e){
   _canerase = false;
 }
 
+function handleDocumentTouchMove(e){
+  erase(findAnnotationAtPoint(e.changedTouches[0].clientX, e.changedTouches[0].clientY));
+}
+
 function handleDocumentMouseMove(e){
+  erase(findAnnotationAtPoint(e.clientX, e.clientY));
+}
+
+function erase(target){
   if(_canerase){
-    let target = findAnnotationAtPoint(e.clientX, e.clientY);
     if(target && target.getAttribute('data-pdf-annotate-userId') == userId){
       let { documentId } = getMetadata(target.parentElement);
       let annotationId = target.getAttribute('data-pdf-annotate-id');
@@ -36,12 +43,18 @@ function handleDocumentMouseMove(e){
 export function enableEraser(){
   userId = PDFJSAnnotate.getStoreAdapter().userId;
   document.addEventListener('mousemove', handleDocumentMouseMove);
-  document.addEventListener('mousedown', handleDocumentMouseDown);
-  document.addEventListener('mouseup', handleDocumentMouseUp);
+  document.addEventListener('mousedown', handleDocumentDown);
+  document.addEventListener('mouseup', handleDocumentUp);
+  document.addEventListener('touchstart', handleDocumentDown);
+  document.addEventListener('touchmove', handleDocumentTouchMove);
+  document.addEventListener('touchend', handleDocumentUp);
 }
 
 export function disableEraser(){
   document.removeEventListener('mousemove', handleDocumentMouseMove);
-  document.removeEventListener('mousedown', handleDocumentMouseDown);
-  document.removeEventListener('mouseup', handleDocumentMouseUp);
+  document.removeEventListener('mousedown', handleDocumentDown);
+  document.removeEventListener('mouseup', handleDocumentUp);
+  document.removeEventListener('touchstart', handleDocumentDown);
+  document.removeEventListener('touchmove', handleDocumentTouchMove);
+  document.removeEventListener('touchend', handleDocumentUp);
 }

src/UI/pen.js

@@ -32,7 +32,7 @@ function handleDocumentPointerdown(e) {
  *
  * @param {Event} e The DOM event to be handled
  */
-function handleDocumentKeyupChrome(e){
+function handleDocumentPointerupChrome(e){
   saveToStorage(e.changedTouches[0].clientX, e.changedTouches[0].clientY);
 }
 
@@ -159,7 +159,7 @@ export function enablePen() {
   if (!isFirefox){
     document.addEventListener('touchstart', handleDocumentPointerdown);
     document.addEventListener('touchmove', handleDocumentPointermoveChrome);
-    document.addEventListener('touchend', handleDocumentKeyupChrome);
+    document.addEventListener('touchend', handleDocumentPointerupChrome);
     document.addEventListener('mousedown', handleDocumentPointerdown);
     document.addEventListener('mousemove', handleDocumentPointermove);
     document.addEventListener('mouseup', handleDocumentPointerup);
@@ -182,7 +182,7 @@ export function disablePen() {
   if (!isFirefox){
     document.removeEventListener('touchstart', handleDocumentPointerdown);
     document.removeEventListener('touchmove', handleDocumentPointermoveChrome);
-    document.removeEventListener('touchend', handleDocumentKeyupChrome);
+    document.removeEventListener('touchend', handleDocumentPointerupChrome);
     document.removeEventListener('mousedown', handleDocumentPointerdown);
     document.removeEventListener('mousemove', handleDocumentPointermove);
     document.removeEventListener('mouseup', handleDocumentPointerup);

web/index.js

@@ -208,7 +208,6 @@ render();
   }
 
   function setPen(size, color) {
-    console.trace();
     let modified = false;
 
     if (penSize !== size) {