fix(Popover): remove old arrow markup, correct enabled className

reactstrap · Nov 28, 2016 · b9d3ea7 · b9d3ea7
1 parent d9e7621
commit b9d3ea7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/src/Popover.js b/src/Popover.js
@@ -23,7 +23,10 @@ const defaultProps = {
 
 const defaultTetherConfig = {
   classPrefix: 'bs-tether',
-  classes: { element: 'popover', enabled: 'open' },
+  classes: {
+    element: false,
+    enabled: 'in'
+  },
   constraints: [
     { to: 'scrollParent', attachment: 'together none' },
     { to: 'window', attachment: 'together none' }
@@ -63,7 +66,7 @@ class Popover extends React.Component {
 
     return (
       <TetherContent
-        arrow="popover"
+        className="popover"
         tether={tetherConfig}
         tetherRef={this.props.tetherRef}
         isOpen={this.props.isOpen}

diff --git a/src/__tests__/Popover.spec.js b/src/__tests__/Popover.spec.js
@@ -68,6 +68,7 @@ describe('Popover', () => {
     expect(isOpen).toBe(false);
 
     expect(wrapper.find(TetherContent).length).toBe(0);
+    expect(document.body.querySelectorAll('.popover.in').length).toBe(0);
     expect(document.getElementsByClassName('popover').length).toBe(0);
     expect(document.getElementsByClassName('popover-inner').length).toBe(0);
     expect(document.getElementsByClassName('popover-title').length).toBe(0);
@@ -80,6 +81,7 @@ describe('Popover', () => {
 
     expect(isOpen).toBe(true);
     expect(wrapper.find(TetherContent).length).toBe(1);
+    expect(document.body.querySelectorAll('.popover.in').length).toBe(1);
     expect(document.getElementsByClassName('popover').length).toBe(1);
     expect(document.getElementsByClassName('popover-inner').length).toBe(1);
     expect(document.getElementsByClassName('popover-title').length).toBe(1);