fix: should re-align correctly when resize window, close: #56

react-component · Jun 15, 2017 · 1b0e962 · 1b0e962
1 parent ee78ba7
commit 1b0e962
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/.gitignore b/.gitignore
@@ -22,5 +22,6 @@ node_modules
 *.css
 build
 lib
+es
 coverage
 yarn.lock
diff --git a/package.json b/package.json
@@ -32,6 +32,7 @@
     "build": "rc-tools run build",
     "gh-pages": "rc-tools run gh-pages",
     "start": "rc-tools run server",
+    "compile": "rc-tools run compile",
     "pub": "rc-tools run pub --babel-runtime",
     "lint": "rc-tools run lint",
     "karma": "rc-test run karma",

diff --git a/src/Popup.js b/src/Popup.js
@@ -27,9 +27,10 @@ class Popup extends Component {
 
   onAlign = (popupDomNode, align) => {
     const props = this.props;
-    const alignClassName = props.getClassNameFromAlign(props.align);
     const currentAlignClassName = props.getClassNameFromAlign(align);
-    if (alignClassName !== currentAlignClassName) {
+    // FIX: https://github.com/react-component/trigger/issues/56
+    // FIX: https://github.com/react-component/tooltip/issues/79
+    if (this.currentAlignClassName !== currentAlignClassName) {
       this.currentAlignClassName = currentAlignClassName;
       popupDomNode.className = this.getClassName(currentAlignClassName);
     }
-Original file line number
+Diff line change
@@ Expand Up / @@ -22,5 +22,6 @@ node_modules @@
     *.css
     build
     lib
+    es
     coverage
     yarn.lock