[changed] Move built components to lib directory

In the past this library has suffered from accidentally pushing this repo directly to npm without the built components. There has been a separate repo to which the built source has been pushed and later deployed from. While the approach is feasible it prevents you from using the `npm link` feature, for easy testing of changes in your own application. This should address concerns like #325, #218, and any others like them that I have missed. In particular this is the fix for an automated setup as mentioned in #292. If you are currently using components directly from this library, then you will need to change your reference path. For example `require('react-bootstrap/Alert')` should now be `require('react-bootstrap/lib/Alert')`. `require('react-bootstrap').Alert` will still work. This applies to both CommonJS and AMD usages.
react-bootstrap · Feb 16, 2015 · 0193046 · 0193046
1 parent 3f1a679
commit 0193046
Show file tree

Hide file tree

Showing 44 changed files with 200 additions and 194 deletions.
diff --git a/.gitignore b/.gitignore
@@ -10,4 +10,5 @@ docs/*.html
 docs/assets/bundle.js
 cjs/*
 amd/*
-ie8/bundle.js
+ie8/bundle.js
+lib/*
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -8,13 +8,13 @@ module.exports = function (grunt) {
         expand: true,
         cwd: 'src/',
         src: ['**/*.js'],
-        dest: 'amd/'
+        dest: 'amd/lib'
       },
       transpiled: {
         expand: true,
         cwd: 'transpiled/',
         src: ['**/*.js'],
-        dest: 'amd/'
+        dest: 'amd/lib'
       }
     },
 
@@ -35,17 +35,17 @@ module.exports = function (grunt) {
             expand: true,
             cwd: 'transpiled/',
             src: ['**/*.js'],
-            dest: 'cjs/'
+            dest: 'lib/'
           },
           {
             expand: true,
             cwd: 'src/',
             src: ['**/*.js'],
-            dest: 'cjs/'
+            dest: 'lib/'
           },
           {
             src: ['**/*'],
-            dest: 'cjs/',
+            dest: 'lib/',
             cwd: 'tools/cjs',
             expand: true
           }
@@ -88,7 +88,7 @@ module.exports = function (grunt) {
 
     clean: {
       transpiled: ['transpiled'],
-      cjs: ['cjs'],
+      cjs: ['lib'],
       amd: ['amd'],
       test: ['test-built']
     },

diff --git a/docs/package.json b/docs/package.json
@@ -24,7 +24,7 @@
   },
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1",
-    "start": "supervisor -w .,../cjs -i node_modules server.js",
+    "start": "supervisor -w .,../lib -i node_modules server.js",
     "build": "node build.js && browserify client.js | uglifyjs -cm 2>/dev/null > ./assets/bundle.js",
     "start-prod": "NODE_ENV=production node server.js"
   },

diff --git a/docs/src/ComponentsPage.js b/docs/src/ComponentsPage.js
@@ -3,10 +3,10 @@
 var React = require('react');
 var fs = require('fs');
 
-var Affix = require('../../cjs/Affix');
-var Nav = require('../../cjs/Nav');
-var SubNav = require('../../cjs/SubNav');
-var NavItem = require('../../cjs/NavItem');
+var Affix = require('../../lib/Affix');
+var Nav = require('../../lib/Nav');
+var SubNav = require('../../lib/SubNav');
+var NavItem = require('../../lib/NavItem');
 
 var NavMain = require('./NavMain');
 var PageHeader = require('./PageHeader');

diff --git a/docs/src/GettingStartedPage.js b/docs/src/GettingStartedPage.js
@@ -34,7 +34,7 @@ var Page = React.createClass({
   $ npm install react-bootstrap\n\
                     '}</code></pre>
                     <pre><code className="js">{'\
-  var Alert = require(\'react-bootstrap/Alert\');\n\
+  var Alert = require(\'react-bootstrap/lib/Alert\');\n\
   // or\n\
   var Alert = require(\'react-bootstrap\').Alert;\
                     '}</code></pre>
@@ -47,7 +47,7 @@ var Page = React.createClass({
   $ bower install react-bootstrap\n\
                     '}</code></pre>
                     <pre><code className="js">{'\
-  var Alert = require(\'react-bootstrap/Alert\');\n\
+  var Alert = require(\'react-bootstrap/lib/Alert\');\n\
   // or\n\
   var Alert = require(\'react-bootstrap\').Alert;\
                     '}</code></pre>

diff --git a/docs/src/NavMain.js b/docs/src/NavMain.js
@@ -2,8 +2,8 @@
 
 var React = require('react');
 var Router = require('react-router-component');
-var Navbar = require('../../cjs/Navbar');
-var Nav = require('../../cjs/Nav');
+var Navbar = require('../../lib/Navbar');
+var Nav = require('../../lib/Nav');
 
 var InternalLink = Router.Link;
 

diff --git a/docs/src/ReactPlayground.js b/docs/src/ReactPlayground.js
@@ -2,45 +2,45 @@ var React = require('react');
 var classSet = require('react/lib/cx');
 var CodeMirror = global.CodeMirror;
 var JSXTransformer = global.JSXTransformer;
-var Accordion = require('../../cjs/Accordion');
-var Alert = require('../../cjs/Alert');
-var Badge = require('../../cjs/Badge');
-var Button = require('../../cjs/Button');
-var ButtonGroup = require('../../cjs/ButtonGroup');
-var ButtonToolbar = require('../../cjs/ButtonToolbar');
-var Carousel = require('../../cjs/Carousel');
-var CarouselItem = require('../../cjs/CarouselItem');
-var Col = require('../../cjs/Col');
-var DropdownButton = require('../../cjs/DropdownButton');
-var Glyphicon = require('../../cjs/Glyphicon');
-var Grid = require('../../cjs/Grid');
-var Input = require('../../cjs/Input');
-var Jumbotron = require('../../cjs/Jumbotron');
-var Label = require('../../cjs/Label');
-var ListGroup = require('../../cjs/ListGroup');
-var ListGroupItem = require('../../cjs/ListGroupItem');
-var Nav = require('../../cjs/Nav');
-var Navbar = require('../../cjs/Navbar');
-var NavItem = require('../../cjs/NavItem');
-var MenuItem = require('../../cjs/MenuItem');
-var Modal = require('../../cjs/Modal');
-var ModalTrigger = require('../../cjs/ModalTrigger');
-var OverlayTrigger = require('../../cjs/OverlayTrigger');
-var OverlayMixin = require('../../cjs/OverlayMixin');
-var PageHeader = require('../../cjs/PageHeader');
-var PageItem = require('../../cjs/PageItem');
-var Pager = require('../../cjs/Pager');
-var Panel = require('../../cjs/Panel');
-var PanelGroup = require('../../cjs/PanelGroup');
-var Popover = require('../../cjs/Popover');
-var ProgressBar = require('../../cjs/ProgressBar');
-var Row = require('../../cjs/Row');
-var SplitButton = require('../../cjs/SplitButton');
-var TabbedArea = require('../../cjs/TabbedArea');
-var Table = require('../../cjs/Table');
-var TabPane = require('../../cjs/TabPane');
-var Tooltip = require('../../cjs/Tooltip');
-var Well = require('../../cjs/Well');
+var Accordion = require('../../lib/Accordion');
+var Alert = require('../../lib/Alert');
+var Badge = require('../../lib/Badge');
+var Button = require('../../lib/Button');
+var ButtonGroup = require('../../lib/ButtonGroup');
+var ButtonToolbar = require('../../lib/ButtonToolbar');
+var Carousel = require('../../lib/Carousel');
+var CarouselItem = require('../../lib/CarouselItem');
+var Col = require('../../lib/Col');
+var DropdownButton = require('../../lib/DropdownButton');
+var Glyphicon = require('../../lib/Glyphicon');
+var Grid = require('../../lib/Grid');
+var Input = require('../../lib/Input');
+var Jumbotron = require('../../lib/Jumbotron');
+var Label = require('../../lib/Label');
+var ListGroup = require('../../lib/ListGroup');
+var ListGroupItem = require('../../lib/ListGroupItem');
+var Nav = require('../../lib/Nav');
+var Navbar = require('../../lib/Navbar');
+var NavItem = require('../../lib/NavItem');
+var MenuItem = require('../../lib/MenuItem');
+var Modal = require('../../lib/Modal');
+var ModalTrigger = require('../../lib/ModalTrigger');
+var OverlayTrigger = require('../../lib/OverlayTrigger');
+var OverlayMixin = require('../../lib/OverlayMixin');
+var PageHeader = require('../../lib/PageHeader');
+var PageItem = require('../../lib/PageItem');
+var Pager = require('../../lib/Pager');
+var Panel = require('../../lib/Panel');
+var PanelGroup = require('../../lib/PanelGroup');
+var Popover = require('../../lib/Popover');
+var ProgressBar = require('../../lib/ProgressBar');
+var Row = require('../../lib/Row');
+var SplitButton = require('../../lib/SplitButton');
+var TabbedArea = require('../../lib/TabbedArea');
+var Table = require('../../lib/Table');
+var TabPane = require('../../lib/TabPane');
+var Tooltip = require('../../lib/Tooltip');
+var Well = require('../../lib/Well');
 
 var IS_MOBILE = typeof navigator !== 'undefined' && (
   navigator.userAgent.match(/Android/i)

diff --git a/ie8/src.js b/ie8/src.js
@@ -1,21 +1,21 @@
 /** @jsx React.DOM */
 var React = require('react');
-var Button = require('../cjs/Button');
-var DropdownButton = require('../cjs/DropdownButton');
-var MenuItem = require('../cjs/MenuItem');
-var Accordion = require('../cjs/Accordion');
-var Panel = require('../cjs/Panel');
-var ButtonToolbar = require('../cjs/ButtonToolbar');
-var OverlayTrigger = require('../cjs/OverlayTrigger');
-var Tooltip = require('../cjs/Tooltip');
-var Alert = require('../cjs/Alert');
-var TabbedArea = require('../cjs/TabbedArea');
-var TabPane = require('../cjs/TabPane');
-var Modal = require('../cjs/Modal');
-var OverlayMixin = require('../cjs/OverlayMixin');
-var Popover = require('../cjs/Popover');
-var Carousel = require('../cjs/Carousel');
-var CarouselItem = require('../cjs/CarouselItem');
+var Button = require('../lib/Button');
+var DropdownButton = require('../lib/DropdownButton');
+var MenuItem = require('../lib/MenuItem');
+var Accordion = require('../lib/Accordion');
+var Panel = require('../lib/Panel');
+var ButtonToolbar = require('../lib/ButtonToolbar');
+var OverlayTrigger = require('../lib/OverlayTrigger');
+var Tooltip = require('../lib/Tooltip');
+var Alert = require('../lib/Alert');
+var TabbedArea = require('../lib/TabbedArea');
+var TabPane = require('../lib/TabPane');
+var Modal = require('../lib/Modal');
+var OverlayMixin = require('../lib/OverlayMixin');
+var Popover = require('../lib/Popover');
+var Carousel = require('../lib/Carousel');
+var CarouselItem = require('../lib/CarouselItem');
 
 var dropdownInstance = (
   <DropdownButton title="Dropdown">

diff --git a/package.json b/package.json
@@ -10,7 +10,12 @@
   "scripts": {
     "build": "./node_modules/.bin/grunt build",
     "test-watch": "./node_modules/.bin/grunt watch 2>&1 >/dev/null & ./node_modules/karma/bin/karma start karma.dev.js",
-    "test": "./node_modules/.bin/grunt build && ./node_modules/karma/bin/karma start karma.ci.js"
+    "test": "./node_modules/.bin/grunt build && ./node_modules/karma/bin/karma start karma.ci.js",
+    "prepublish": "./node_modules/.bin/grunt build"
+  },
+  "main": "lib/main.js",
+  "directories": {
+    "lib": "lib/"
   },
   "keywords": [
     "react",

diff --git a/test/AlertSpec.jsx b/test/AlertSpec.jsx
@@ -2,7 +2,7 @@
 
 var React          = require('react');
 var ReactTestUtils = require('react/lib/ReactTestUtils');
-var Alert          = require('../cjs/Alert');
+var Alert          = require('../lib/Alert');
 
 describe('Alert', function () {
   it('Should output a alert with message', function () {
@@ -73,4 +73,4 @@ describe('Alert', function () {
     );
     assert.ok(instance.getDOMNode().className.match(/\balert-danger\b/));
   });
-});
+});
diff --git a/test/BadgeSpec.jsx b/test/BadgeSpec.jsx
@@ -2,7 +2,7 @@
 
 var React          = require('react');
 var ReactTestUtils = require('react/lib/ReactTestUtils');
-var Badge          = require('../cjs/Badge');
+var Badge          = require('../lib/Badge');
 
 describe('Badge', function () {
   it('Should output a badge with content', function () {

diff --git a/test/BootstrapMixinSpec.jsx b/test/BootstrapMixinSpec.jsx
@@ -2,7 +2,7 @@
 
 var React          = require('react');
 var ReactTestUtils = require('react/lib/ReactTestUtils');
-var BootstrapMixin = require('../cjs/BootstrapMixin');
+var BootstrapMixin = require('../lib/BootstrapMixin');
 
 var Component;
 
@@ -189,4 +189,4 @@ describe('BootstrapMixin', function () {
       assert.deepEqual(instance.getBsClassSet(), {'btn': true, 'btn-xs': true});
     });
   });
-});
+});
diff --git a/test/ButtonGroupSpec.jsx b/test/ButtonGroupSpec.jsx
@@ -2,8 +2,8 @@
 
 var React          = require('react');
 var ReactTestUtils = require('react/lib/ReactTestUtils');
-var ButtonGroup    = require('../cjs/ButtonGroup');
-var Button         = require('../cjs/Button');
+var ButtonGroup    = require('../lib/ButtonGroup');
+var Button         = require('../lib/Button');
 
 describe('ButtonGroup', function () {
   it('Should output a button group', function () {
@@ -50,4 +50,4 @@ describe('ButtonGroup', function () {
     );
     assert.ok(instance.getDOMNode().className.match(/\bbtn-group-justified\b/));
   });
-});
+});
diff --git a/test/ButtonSpec.jsx b/test/ButtonSpec.jsx
@@ -2,7 +2,7 @@
 
 var React          = require('react');
 var ReactTestUtils = require('react/lib/ReactTestUtils');
-var Button         = require('../cjs/Button');
+var Button         = require('../lib/Button');
 
 describe('Button', function () {
   it('Should output a button', function () {

diff --git a/test/ButtonToolbarSpec.jsx b/test/ButtonToolbarSpec.jsx
@@ -2,9 +2,9 @@
 
 var React          = require('react');
 var ReactTestUtils = require('react/lib/ReactTestUtils');
-var ButtonToolbar  = require('../cjs/ButtonToolbar');
-var ButtonGroup    = require('../cjs/ButtonGroup');
-var Button         = require('../cjs/Button');
+var ButtonToolbar  = require('../lib/ButtonToolbar');
+var ButtonGroup    = require('../lib/ButtonGroup');
+var Button         = require('../lib/Button');
 
 describe('ButtonToolbar', function () {
   it('Should output a button toolbar', function () {
@@ -21,4 +21,4 @@ describe('ButtonToolbar', function () {
     assert.ok(instance.getDOMNode().className.match(/\bbtn-toolbar\b/));
     assert.equal(instance.getDOMNode().getAttribute('role'), 'toolbar');
   });
-});
+});
diff --git a/test/CarouselSpec.jsx b/test/CarouselSpec.jsx
@@ -2,8 +2,8 @@
 
 var React           = require('react');
 var ReactTestUtils  = require('react/lib/ReactTestUtils');
-var Carousel        = require('../cjs/Carousel');
-var CarouselItem    = require('../cjs/CarouselItem');
+var Carousel        = require('../lib/Carousel');
+var CarouselItem    = require('../lib/CarouselItem');
 
 describe('Carousel', function () {
   it('Should show the correct item', function () {
@@ -73,4 +73,4 @@ describe('Carousel', function () {
       )[0]
     );
   });
-});
+});
diff --git a/test/CustomPropTypesSpec.jsx b/test/CustomPropTypesSpec.jsx
@@ -2,7 +2,7 @@
 
 var React           = require('react');
 var ReactTestUtils  = require('react/lib/ReactTestUtils');
-var CustomPropTypes = require('../cjs/utils/CustomPropTypes');
+var CustomPropTypes = require('../lib/utils/CustomPropTypes');
 
 describe('CustomPropTypes', function () {
 

diff --git a/test/DropdownButtonSpec.jsx b/test/DropdownButtonSpec.jsx
@@ -2,10 +2,10 @@
 
 var React          = require('react');
 var ReactTestUtils = require('react/lib/ReactTestUtils');
-var DropdownButton = require('../cjs/DropdownButton');
-var MenuItem       = require('../cjs/MenuItem');
-var DropdownMenu   = require('../cjs/DropdownMenu');
-var Button         = require('../cjs/Button');
+var DropdownButton = require('../lib/DropdownButton');
+var MenuItem       = require('../lib/MenuItem');
+var DropdownMenu   = require('../lib/DropdownMenu');
+var Button         = require('../lib/Button');
 
 describe('DropdownButton', function () {
   var instance;
@@ -203,4 +203,4 @@ describe('DropdownButton', function () {
     var carets = button.getElementsByClassName('caret');
     assert.equal(carets.length, 0);
   });
-});
+});
diff --git a/test/DropdownMenuSpec.jsx b/test/DropdownMenuSpec.jsx
@@ -2,8 +2,8 @@
 
 var React          = require('react');
 var ReactTestUtils = require('react/lib/ReactTestUtils');
-var DropdownMenu   = require('../cjs/DropdownMenu');
-var MenuItem       = require('../cjs/MenuItem');
+var DropdownMenu   = require('../lib/DropdownMenu');
+var MenuItem       = require('../lib/MenuItem');
 
 describe('DropdownMenu', function () {
   it('Should render menu correctly', function () {
@@ -100,4 +100,4 @@ describe('DropdownMenu', function () {
     ReactTestUtils.findRenderedDOMComponentWithTag(menuItems[1], 'a').getDOMNode()
       .dispatchEvent(evt);
   });
-});
+});