Fix: AnnotationPopover CSS to not hide popover svgs (#304)

box · Nov 30, 2018 · 83ca5b7 · 83ca5b7
1 parent 4f0418e
commit 83ca5b7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 18 deletions.
diff --git a/src/AnnotationThread.js b/src/AnnotationThread.js
@@ -87,8 +87,6 @@ class AnnotationThread extends EventEmitter {
         // $FlowFixMe
         this.renderAnnotationPopover = this.renderAnnotationPopover.bind(this);
         // $FlowFixMe
-        this.handleBlur = this.handleBlur.bind(this);
-        // $FlowFixMe
         this.onCommentClick = this.onCommentClick.bind(this);
         // $FlowFixMe
         this.save = this.save.bind(this);
@@ -453,7 +451,6 @@ class AnnotationThread extends EventEmitter {
         }
 
         this.element.addEventListener('click', this.renderAnnotationPopover);
-        this.element.addEventListener('blur', this.handleBlur);
     }
 
     /**
@@ -467,16 +464,6 @@ class AnnotationThread extends EventEmitter {
         }
 
         this.element.removeEventListener('click', this.renderAnnotationPopover);
-        this.element.removeEventListener('blur', this.handleBlur);
-    }
-
-    /**
-     * Called when the annotation element loses focus
-     *
-     * @return {void}
-     */
-    handleBlur() {
-        this.toggleFlippedThreadEl();
     }
 
     /**

diff --git a/src/components/AnnotationPopover/AnnotationPopover.scss b/src/components/AnnotationPopover/AnnotationPopover.scss
@@ -75,15 +75,17 @@
     }
 
     .ba-popover-flipped {
-        path,
-        rect {
-            display: none;
-        }
-
         .ba-popover-caret {
             bottom: 0;
             transform: rotate(180deg);
         }
+
+        &.ba-point-annotation-marker {
+            path,
+            rect {
+                display: none;
+            }
+        }
     }
 
     // MOBILE & TABLET CSS