Merge pull request #41 from ndelangen/fix/37

FIX yjose/reactjs-popup#37
web3gru · Sep 29, 2018 · f5375d6 · f5375d6
2 parents 2d69925 + 6968380
commit f5375d6
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/Popup.js b/src/Popup.js
@@ -252,6 +252,11 @@ export default class Popup extends React.PureComponent {
     const overlay = this.state.isOpen && !(on.indexOf("hover") >= 0);
     const ovStyle = modal ? styles.overlay.modal : styles.overlay.tooltip;
     return [
+      !!this.props.trigger && (
+        <Ref innerRef={this.setTriggerRef} key="R">
+          {this.renderTrigger()}
+        </Ref>
+      ),
       this.state.isOpen && (
         <div
           key="H"
@@ -270,11 +275,6 @@ export default class Popup extends React.PureComponent {
         </div>
       ),
       this.state.isOpen && !modal && this.renderContent(),
-      !!this.props.trigger && (
-        <Ref innerRef={this.setTriggerRef} key="R">
-          {this.renderTrigger()}
-        </Ref>
-      )
     ];
   }
 }

diff --git a/stories/src/BoundedTooltip.js b/stories/src/BoundedTooltip.js
@@ -19,12 +19,12 @@ export default class BoundedTooltip extends React.Component {
         position={this.props.position}
         //open={this.state.open}
         closeOnDocumentClick
-        keepTooltipInside={false}
+        keepTooltipInside={true}
         // onClose={this.closeModal}
       >
         {close => (
           <div className="modal">
-            <a className="close" onClick={close}>
+            <a className="close" onClick={close} href="#!">
               &times; remove
             </a>
             Lorem ipsum dolor sit amet, consectur adipisicing elit. Beatae magni