[removed] Nav right prop in favor of pullRight

kyletsang · Nov 15, 2015 · 0be007f · 0be007f
1 parent 5dbafd3
commit 0be007f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/Nav.js b/src/Nav.js
@@ -18,11 +18,12 @@ class Nav extends React.Component {
     classes[tbsUtils.prefix(this.props, 'justified')] = this.props.justified;
 
 
-    if (this.props.navbar) {
+    if (isNavbar) {
       let bsClass = this.context.$bs_navbar_bsClass || 'navbar';
+      const navbarRight = this.props.right != null ? this.props.right : this.props.pullRight;
 
       classes[tbsUtils.prefix({ bsClass }, 'nav')] = true;
-      classes[tbsUtils.prefix({ bsClass }, 'right')] = isNavbar;
+      classes[tbsUtils.prefix({ bsClass }, 'right')] = navbarRight;
     } else {
       classes['pull-right'] = this.props.pullRight;
     }
@@ -119,7 +120,7 @@ Nav.propTypes = {
   navbar: React.PropTypes.bool,
   eventKey: React.PropTypes.any,
   pullRight: React.PropTypes.bool,
-  right: React.PropTypes.bool
+  right: deprecated(React.PropTypes.bool, 'Use the `pullRight` prop instead')
 };
 
 Nav.contextTypes = {

diff --git a/test/NavbarSpec.js b/test/NavbarSpec.js
@@ -186,7 +186,7 @@ describe('Navbar', () => {
     expect(collapse.context.$bs_navbar_navExpanded).to.equal(true);
   });
 
-  it.only('Should wire the toggle to the collapse', () => {
+  it('Should wire the toggle to the collapse', () => {
     let instance = ReactTestUtils.renderIntoDocument(
       <Navbar>
         <Navbar.Header>