Repository: react-materialize/react-materialize (js)

Issue:
Chips with "close" prop is broken in 3.0.0
### Description

Chips with "close" prop missing actual "close" class.

### Steps to Reproduce

1. Create a Chip with close prop
2. Code example from story:
```jsx
stories.add('with close', () => (
  <Chip close>Jane Doe</Chip>
));
```

**Expected behavior:** Proper styles for close icon
![image](https://user-images.githubusercontent.com/1886857/54082818-c59fa080-4323-11e9-9d9b-57c5bdc0525e.png)
![image](https://user-images.githubusercontent.com/1886857/54082822-d05a3580-4323-11e9-9007-36df3a2d07c3.png)

**Actual behavior:** Wrong style because of missing "close" class on <i>
![image](https://user-images.githubusercontent.com/1886857/54082781-7e191480-4323-11e9-9103-a90e65bc78a6.png)
![image](https://user-images.githubusercontent.com/1886857/54082811-b7518480-4323-11e9-9b36-3dde5a4011c1.png)


### Versions

**react-materialize:** 3.0.0-beta.0
**materialize-css:** 1.0.0
**react:** 16.8

#### Thanks for helping out!


Pull request:
Fix close icon on Chip
Fixes #769

Patch:
diff --git a/src/Chip.js b/src/Chip.js
index 3b6e2df..f86a76f 100644
--- a/src/Chip.js
+++ b/src/Chip.js
@@ -29,7 +29,7 @@ class Chip extends Component {
     let chipContent = (
       <React.Fragment>
         {children}
-        {close && <Icon>close</Icon>}
+        {close && <Icon className="close">close</Icon>}
       </React.Fragment>
     );
 
diff --git a/src/SideNav.js b/src/SideNav.js
index 1354d03..57c19d7 100644
--- a/src/SideNav.js
+++ b/src/SideNav.js
@@ -33,12 +33,7 @@ class SideNav extends Component {
     return (
       <Fragment>
         {this.renderTrigger()}
-        <ul
-          ref={el => (this._sidenav = el)}
-          id={this.id}
-          className={classNames}
-          {...props}
-        >
+        <ul id={this.id} className={classNames} {...props}>
           {children}
         </ul>
       </Fragment>
diff --git a/stories/@Welcome.stories.js b/stories/@Welcome.stories.js
index 181082f..9b8e98b 100644
--- a/stories/@Welcome.stories.js
+++ b/stories/@Welcome.stories.js
@@ -27,7 +27,7 @@ stories.add('Welcome', () => (
       <img
         src="https://camo.githubusercontent.com/4533a6b0f2046a75aa9b965477287088b86e3897/687474703a2f2f696d672e736869656c64732e696f2f6e706d2f762f72656163742d6d6174657269616c697a652e7376673f7374796c653d666c6174"
         alt="npm version"
-        datacanonicalsrc="http://img.shields.io/npm/v/react-materialize.svg?style=flat"
+        dataCanonicalSrc="http://img.shields.io/npm/v/react-materialize.svg?style=flat"
         style={{ maxWidth: '100%' }}
       />
     </a>
diff --git a/stories/Parallax.stories.js b/stories/Parallax.stories.js
deleted file mode 100644
index 90b8bf3..0000000
--- a/stories/Parallax.stories.js
+++ /dev/null
@@ -1,24 +0,0 @@
-import React from 'react';
-import { storiesOf } from '@storybook/react';
-import Parallax from '../src/Parallax';
-
-const stories = storiesOf('Javascript|Parallax', module);
-
-stories.addParameters({
-  info: {
-    text: `Parallax is an effect where the background content or image in this case, is moved at a different speed than the foreground content while scrolling. Check out the demo to get a better idea of it.`
-  }
-});
-
-stories.add('Default', () => (
-  <div>
-    <Parallax imageSrc="http://materializecss.com/images/parallax1.jpg"/>
-    <div className="section white">
-      <div className="row container">
-        <h2 className="header">Parallax</h2>
-        <p className="grey-text text-darken-3 lighten-3">Parallax is an effect where the background content or image in this case, is moved at a different speed than the foreground content while scrolling.</p>
-      </div>
-    </div>
-    <Parallax imageSrc="http://materializecss.com/images/parallax2.jpg"/>
-  </div>
-));
diff --git a/stories/Preloader.stories.js b/stories/Preloader.stories.js
deleted file mode 100644
index 1c41548..0000000
--- a/stories/Preloader.stories.js
+++ /dev/null
@@ -1,41 +0,0 @@
-import React from 'react';
-import { storiesOf } from '@storybook/react';
-import Row from '../src/Row';
-import Col from '../src/Col';
-import ProgressBar from '../src/ProgressBar';
-import Preloader from '../src/Preloader';
-import { number, withKnobs } from '@storybook/addon-knobs';
-
-const stories = storiesOf('Components|Preloader', module);
-stories.addDecorator(withKnobs);
-
-stories.addParameters({
-  info: {
-    text: `If you have content that will take a long time to load, you should give the user feedback. For this reason we provide a number activity + progress indicators.`
-  }
-});
-
-stories.add('Default', () => (
-  <Row>
-    <Col s={12}>
-      <ProgressBar progress={number('progress', 70)} />
-    </Col>
-    <Col s={12}>
-      <ProgressBar />
-    </Col>
-  </Row>
-));
-
-stories.add('Circular', () => (
-  <Row>
-    <Col s={4}>
-      <Preloader size="big" />
-    </Col>
-    <Col s={4}>
-      <Preloader flashing />
-    </Col>
-    <Col s={4}>
-      <Preloader size="small" />
-    </Col>
-  </Row>
-));


Test patch:
diff --git a/test/Chip.spec.js b/test/Chip.spec.js
index e5f513e..ef69559 100644
--- a/test/Chip.spec.js
+++ b/test/Chip.spec.js
@@ -14,6 +14,7 @@ describe('<Chip />', () => {
 
   test('accepts a close prop', () => {
     wrapper = shallow(<Chip close />);
+    expect(wrapper.find(Icon).prop('className')).toBe('close');
     expect(wrapper.find(Icon)).toHaveLength(1);
   });
 
diff --git a/test/__snapshots__/Chip.spec.js.snap b/test/__snapshots__/Chip.spec.js.snap
index bf1b804..f7f1d55 100644
--- a/test/__snapshots__/Chip.spec.js.snap
+++ b/test/__snapshots__/Chip.spec.js.snap
@@ -9,7 +9,9 @@ exports[`<Chip /> accepts children 1`] = `
   >
     Child
   </p>
-  <Icon>
+  <Icon
+    className="close"
+  >
     close
   </Icon>
 </div>