fix(Popup): apply RAF to pin Popup to trigger component

Semantic-Org · May 8, 2018 · a8c3d70 · a8c3d70
1 parent 9bc4905
commit a8c3d70
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/src/modules/Popup/Popup.js b/src/modules/Popup/Popup.js
@@ -308,6 +308,7 @@ export default class Popup extends Component {
     const { hideOnScroll } = this.props
 
     if (hideOnScroll) eventStack.sub('scroll', this.hideOnScroll, { target: window })
+    this.setPosition()
     _.invoke(this.props, 'onMount', e, this.props)
   }
 
@@ -332,6 +333,14 @@ export default class Popup extends Component {
     }
   }
 
+  setPosition = () => {
+    if (this.triggerRef) {
+      this.setPopupStyle(this.props.position)
+    }
+
+    this.animationRequestId = requestAnimationFrame(this.setPosition)
+  }
+
   render() {
     const {
       basic,