Chore: Cleaning up annotations code comments (#206)

box · Jul 10, 2017 · 9e042c1 · 9e042c1
1 parent 5b8539f
commit 9e042c1
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 9 deletions.
diff --git a/src/lib/annotations/AnnotationDialog.js b/src/lib/annotations/AnnotationDialog.js
@@ -161,7 +161,6 @@ const CLASS_ANIMATE_DIALOG = 'bp-animate-show-dialog';
     /**
      * Hides the dialog.
      *
-     * @param {boolean} [noDelay] - Whether or not to have a timeout delay
      * @return {void}
      */
     hide() {
@@ -319,7 +318,7 @@ const CLASS_ANIMATE_DIALOG = 'bp-animate-show-dialog';
 
         const key = decodeKeydown(event);
         if (key === 'Escape') {
-            this.hide(); // hide without delay
+            this.hide();
         } else {
             const dataType = annotatorUtil.findClosestDataType(event.target);
             if (dataType === 'reply-textarea') {

diff --git a/src/lib/annotations/Annotator.js b/src/lib/annotations/Annotator.js
@@ -202,9 +202,6 @@ import './Annotator.scss';
         }
 
         // Hide create annotations button if image is rotated
-        // TODO(@spramod) actually adjust getLocationFromEvent method
-        // in annotator to get correct location rather than disabling
-        // the creation of annotations on rotated images
         const annotateButton = this.previewUI.getAnnotateButton();
 
         if (rotationAngle !== 0) {

diff --git a/src/lib/viewers/image/ImageBaseViewer.js b/src/lib/viewers/image/ImageBaseViewer.js
@@ -317,10 +317,6 @@ const CSS_CLASS_PANNABLE = 'pannable';
         event.stopPropagation();
     }
 
-    //----------------------------------------------------------------------------------------------
-    // Annotations. TODO(jholdstock|spramod) Refactor out all annotations code from viewers
-    //----------------------------------------------------------------------------------------------
-
     /**
      * Disables viewer controls
      *