Very cool, great to see this progressing. playTrack can be a little more concise:
playTrack
let playTrack = index => dispatch(switch (playing) { | Playing(idx) when index === idx => MusicPlayer.PauseTrack | _ => MusicPlayer.PlayTrack(index) });
This captures the fact that we pause only when the current index is playing, and play otherwise; and that we need to dispatch from both branches.
Are you sure you want to hide this comment? It will become hidden in your post, but will still be visible via the comment's permalink.
Hide child comments as well
Confirm
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
Very cool, great to see this progressing.
playTrack
can be a little more concise:This captures the fact that we pause only when the current index is playing, and play otherwise; and that we need to dispatch from both branches.