frogfood / bwa-u3-jammming-prj Goto Github PK

the handleNameChange method in PlayList.js needs to be set to props like so

  handleNameChange(event) {
    this.props.onNameChange(event.target.value);
  }

it is recommend to ship the code with a package.json file in the root folder. That way it is easier for someone else to download and load the dependencies the project needs.

in Track.js the function should not be named render because reactjs has a built-in function called render. We will change the name to renderAction().

We will call this on line 33
https://github.com/frogfood/bwa-u3-jammming-prj/blob/master/src/Components/Track/Track.js#L33
instead of
{addRemove}
we will use
{this.renderAction()}

The project may have seemed to have been rushed a bit. However, the coding still came out well. I would suggest going over props and state in depth. here is ref that will help
https://facebook.github.io/react-vr/docs/components-props-and-state.html

Here is a more elegant way to remove tracks using the filter method

  removeTrack(track) {
    let tracks = this.state.playlistTracks;
    tracks = tracks.filter(currentTrack => currentTrack.id !== track.id);

    this.setState({playlistTracks: tracks});
  }