Merge pull request #36 from mesosphere/cleanup/DCOS-3190-consistent-default-classes

Do default classes the same way

Author: Kenny Tran

src/List/List.js

@@ -1,5 +1,4 @@
 import React, {PropTypes} from 'react/addons';
-import classNames from 'classnames';
 
 import ListItem from './ListItem';
 
@@ -40,18 +39,10 @@ export default class List extends React.Component {
   }
 
   render() {
-    let defaultClasses = [
-      'list'
-    ];
-
-    let classes = classNames(
-      defaultClasses.concat(this.props.className.split(' '))
-    );
-
     let Tag = this.props.tag;
 
     return (
-      <Tag {...this.props} className={classes}>
+      <Tag {...this.props} className={this.props.className}>
         {this.getListItems(this.props.items)}
       </Tag>
     );
@@ -63,15 +54,19 @@ List.defaultProps = {
   attributes: {
     className: ''
   },
-  className: '',
-  tag: 'ul'
+  tag: 'ul',
+
+  // Default classes.
+  className: 'list'
 };
 
 List.propTypes = {
   attributes: PropTypes.object,
-  className: PropTypes.string,
   items: PropTypes.array.isRequired,
   tag: PropTypes.string,
   transition: PropTypes.bool,
-  transitionName: PropTypes.string
+  transitionName: PropTypes.string,
+
+  // Classes.
+  className: PropTypes.string
 };

src/Modal/DefaultModalClasses.js

@@ -247,7 +247,7 @@ ModalContents.defaultProps = {
   subHeader: null,
   titleText: '',
 
-  // Classes
+  // Default classes.
   backdropClass: 'fade in modal-backdrop',
   bodyClass: 'modal-content',
   closeButtonClass: 'modal-close',
@@ -276,7 +276,7 @@ ModalContents.propTypes = {
   subHeader: PropTypes.node,
   titleText: PropTypes.string,
 
-  // Classes
+  // Classes.
   backdropClass: PropTypes.string,
   bodyClass: PropTypes.string,
   closeButtonClass: PropTypes.string,

src/Modal/ModalContents.js

@@ -138,7 +138,7 @@ SidePanelContents.defaultProps = {
   onClose: () => {},
   open: false,
 
-  // Classes
+  // Default classes.
   backdropClass: 'side-panel-backdrop',
   bodyClass: 'side-panel-content container container-scrollable container-fluid container-fluid-narrow',
   containerClass: 'side-panel-container',
@@ -154,7 +154,7 @@ SidePanelContents.propTypes = {
   onClose: PropTypes.func,
   open: PropTypes.bool,
 
-  // Classes
+  // Classes.
   backdropClass: PropTypes.string,
   bodyClass: PropTypes.string,
   containerClass: PropTypes.string,