Skip to content

Commit

Permalink
prettier all the sources 💅
Browse files Browse the repository at this point in the history
  • Loading branch information
souporserious committed Sep 28, 2017
1 parent 78e4f4c commit f89327f
Show file tree
Hide file tree
Showing 24 changed files with 169 additions and 140 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,7 @@
## CHANGELOG
### 0.6.4
Return `play` in HTML5 method so we can react to the promise

### 0.6.3
Update ALL imports of PropTypes [#37](https://github.com/souporserious/react-media-player/pull/37)

Expand Down
22 changes: 12 additions & 10 deletions package.json
Original file line number Diff line number Diff line change
@@ -1,20 +1,21 @@
{
"name": "react-media-player",
"version": "0.6.3",
"version": "0.6.4",
"description": "React media player.",
"main": "lib/react-media-player.js",
"files": [
"dist",
"lib"
],
"files": ["dist", "lib"],
"scripts": {
"build:lib": "babel src --out-dir lib",
"build": "npm run build:lib && NODE_ENV=production webpack --config webpack.prod.config.js",
"dev": "webpack-dev-server --inline --hot --progress --colors --host 0.0.0.0 --devtool eval",
"postbuild": "NODE_ENV=production TARGET=minify webpack --config webpack.prod.config.js",
"build":
"npm run build:lib && NODE_ENV=production webpack --config webpack.prod.config.js",
"dev":
"webpack-dev-server --inline --hot --progress --colors --host 0.0.0.0 --devtool eval",
"postbuild":
"NODE_ENV=production TARGET=minify webpack --config webpack.prod.config.js",
"prebuild": "rm -rf dist && mkdir dist",
"prepublish": "npm run build",
"deploy": "NODE_ENV=production TARGET=minify webpack && git-directory-deploy --directory example --branch gh-pages"
"deploy":
"NODE_ENV=production TARGET=minify webpack && git-directory-deploy --directory example --branch gh-pages"
},
"repository": {
"type": "git",
Expand All @@ -29,7 +30,8 @@
"player",
"playlist"
],
"author": "Travis Arnold <travis@souporserious.com> (http://souporserious.com)",
"author":
"Travis Arnold <travis@souporserious.com> (http://souporserious.com)",
"license": "MIT",
"bugs": {
"url": "https://github.com/souporserious/react-media-player/issues"
Expand Down
40 changes: 20 additions & 20 deletions src/Media.jsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import React, { Component, Children } from 'react'
import PropTypes from 'prop-types';
import PropTypes from 'prop-types'
import ReactDOM, { findDOMNode } from 'react-dom'
import contextTypes from './context-types'
import requestFullscreen from './utils/request-fullscreen'
Expand All @@ -20,7 +20,7 @@ const MEDIA_EVENTS_KEYS = Object.keys(MEDIA_EVENTS)

class Media extends Component {
static propTypes = {
children: PropTypes.oneOfType([PropTypes.func, PropTypes.node]).isRequired
children: PropTypes.oneOfType([PropTypes.func, PropTypes.node]).isRequired,
}

static childContextTypes = contextTypes
Expand All @@ -33,7 +33,7 @@ class Media extends Component {
isLoading: true,
isPlaying: false,
isMuted: false,
isFullscreen: false
isFullscreen: false,
}

_playerProps = {}
Expand All @@ -45,11 +45,11 @@ class Media extends Component {
_mediaSetters: {
setPlayer: this._setPlayer,
setPlayerProps: this._setPlayerProps,
setPlayerState: this._setPlayerState
setPlayerState: this._setPlayerState,
},
_mediaGetters: {
getPlayerEvents: this._getPlayerEvents()
}
getPlayerEvents: this._getPlayerEvents(),
},
}
}

Expand All @@ -74,7 +74,7 @@ class Media extends Component {
muteUnmute: this.muteUnmute,
setVolume: this.setVolume,
addVolume: this.addVolume,
fullscreen: this.fullscreen
fullscreen: this.fullscreen,
}
}

Expand All @@ -85,7 +85,7 @@ class Media extends Component {
const stateKey = MEDIA_EVENTS[key]
const propCallback = this._playerProps[key]

events[key] = (val) => {
events[key] = val => {
if (stateKey) {
this.setState({ [stateKey]: val })
}
Expand All @@ -97,15 +97,15 @@ class Media extends Component {
return events
}

_setPlayer = (component) => {
_setPlayer = component => {
this._player = component
}

_setPlayerProps = (props) => {
_setPlayerProps = props => {
this._playerProps = props
}

_setPlayerState = (state) => {
_setPlayerState = state => {
this.setState(state)
}

Expand All @@ -129,14 +129,14 @@ class Media extends Component {
this._player.stop()
}

seekTo = (currentTime) => {
seekTo = currentTime => {
this._player.seekTo(currentTime)
this.setState({ currentTime })
}

skipTime = (amount) => {
skipTime = amount => {
const { currentTime, duration } = this.state
let newTime = (currentTime + amount)
let newTime = currentTime + amount

if (newTime < 0) {
newTime = 0
Expand All @@ -147,12 +147,12 @@ class Media extends Component {
this.seekTo(newTime)
}

mute = (isMuted) => {
mute = isMuted => {
if (isMuted) {
this._lastVolume = this.state.volume
this._player.setVolume(0)
} else {
const volume = (this._lastVolume > 0) ? this._lastVolume : 0.1
const volume = this._lastVolume > 0 ? this._lastVolume : 0.1
this._player.setVolume(volume)
}
this._player.mute(isMuted)
Expand All @@ -162,8 +162,8 @@ class Media extends Component {
this.mute(!this.state.isMuted)
}

setVolume = (volume) => {
const isMuted = (volume <= 0)
setVolume = volume => {
const isMuted = volume <= 0

if (isMuted !== this.state.isMuted) {
this.mute(isMuted)
Expand All @@ -174,8 +174,8 @@ class Media extends Component {
this._player.setVolume(volume)
}

addVolume = (amount) => {
let newVolume = (this.state.volume + (amount * 0.01))
addVolume = amount => {
let newVolume = this.state.volume + amount * 0.01

if (newVolume < 0) {
newVolume = 0
Expand Down
46 changes: 27 additions & 19 deletions src/Player.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,13 @@ class Player extends Component {
vendor: PropTypes.oneOf(['video', 'audio', 'youtube', 'vimeo']),
defaultCurrentTime: PropTypes.number,
defaultVolume: PropTypes.number,
defaultMuted: PropTypes.bool
defaultMuted: PropTypes.bool,
}

static defaultProps = {
defaultCurrentTime: -1,
defaultVolume: 1,
defaultMuted: false
defaultMuted: false,
}

static contextTypes = contextTypes
Expand All @@ -40,7 +40,7 @@ class Player extends Component {
progress: 0,
duration: 0,
isLoading: true,
isPlaying: false
isPlaying: false,
})
}
}
Expand All @@ -49,7 +49,7 @@ class Player extends Component {
return this._component && this._component.instance
}

_setPlayer = (component) => {
_setPlayer = component => {
this.context._mediaSetters.setPlayer(component)
this._component = component
}
Expand All @@ -71,7 +71,7 @@ class Player extends Component {
this._defaultsSet = true
}

_setLoading = (isLoading) => {
_setLoading = isLoading => {
this.context._mediaSetters.setPlayerState({ isLoading })
}

Expand Down Expand Up @@ -114,22 +114,30 @@ class Player extends Component {
}

render() {
const { src, vendor: _vendor, autoPlay, onReady, onEnded, defaultCurrentTime, defaultVolume, defaultMuted, ...extraProps } = this.props
const {
src,
vendor: _vendor,
autoPlay,
onReady,
onEnded,
defaultCurrentTime,
defaultVolume,
defaultMuted,
...extraProps
} = this.props
const { vendor, component } = getVendor(src, _vendor)

return (
createElement(component, {
ref: this._setPlayer,
src,
vendor,
autoPlay,
isLoading: this._setLoading,
onReady: this._handleOnReady,
onEnded: this._handleOnEnded,
extraProps,
...this.context._mediaGetters.getPlayerEvents,
})
)
return createElement(component, {
ref: this._setPlayer,
src,
vendor,
autoPlay,
isLoading: this._setLoading,
onReady: this._handleOnReady,
onEnded: this._handleOnEnded,
extraProps,
...this.context._mediaGetters.getPlayerEvents,
})
}
}

Expand Down
4 changes: 2 additions & 2 deletions src/context-types.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import PropTypes from 'prop-types';
import PropTypes from 'prop-types'

export default {
media: PropTypes.object,
_mediaSetters: PropTypes.object,
_mediaGetters: PropTypes.object
_mediaGetters: PropTypes.object,
}
2 changes: 1 addition & 1 deletion src/controls/Fullscreen.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ class Fullscreen extends Component {
style={style}
onClick={this._handleFullscreen}
>
{ media.isFullscreen ? 'Exit Fullscreen' : 'Fullscreen' }
{media.isFullscreen ? 'Exit Fullscreen' : 'Fullscreen'}
</button>
)
}
Expand Down
2 changes: 1 addition & 1 deletion src/controls/MuteUnmute.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ class MuteUnmute extends Component {
style={style}
onClick={this._handleMuteUnmute}
>
{ media.isMuted ? 'Unmute' : 'Mute' }
{media.isMuted ? 'Unmute' : 'Mute'}
</button>
)
}
Expand Down
2 changes: 1 addition & 1 deletion src/controls/PlayPause.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ class PlayPause extends Component {
style={style}
onClick={this._handlePlayPause}
>
{ media.isPlaying ? 'Pause' : 'Play' }
{media.isPlaying ? 'Pause' : 'Play'}
</button>
)
}
Expand Down
12 changes: 7 additions & 5 deletions src/controls/SeekBar.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,10 @@ class SeekBar extends Component {
_onChangeUsed = false

shouldComponentUpdate({ media }) {
return this.props.media.currentTime !== media.currentTime ||
this.props.media.duration !== media.duration
return (
this.props.media.currentTime !== media.currentTime ||
this.props.media.duration !== media.duration
)
}

_handleMouseDown = () => {
Expand Down Expand Up @@ -41,15 +43,15 @@ class SeekBar extends Component {
<input
type="range"
step="any"
max={(duration).toFixed(4)}
max={duration.toFixed(4)}
value={currentTime}
onMouseDown={this._handleMouseDown}
onMouseUp={this._handleMouseUp}
onChange={this._handleChange}
className={className}
style={{
backgroundSize: (currentTime * 100 / duration) + '% 100%',
...style
backgroundSize: currentTime * 100 / duration + '% 100%',
...style,
}}
/>
)
Expand Down
4 changes: 2 additions & 2 deletions src/controls/Volume.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,8 @@ class Volume extends Component {
onChange={this._handleChange}
className={className}
style={{
backgroundSize: (volume * 100 / 1) + '% 100%',
...style
backgroundSize: volume * 100 / 1 + '% 100%',
...style,
}}
/>
)
Expand Down
7 changes: 1 addition & 6 deletions src/decorators/with-media-props.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,7 @@ export default function withMediaProps(MediaComponent) {
static contextTypes = contextTypes

render() {
return (
<MediaComponent
{...this.props}
media={this.context.media}
/>
)
return <MediaComponent {...this.props} media={this.context.media} />
}
}
}
6 changes: 2 additions & 4 deletions src/utils/exit-fullscreen.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,7 @@ export default (() => {
'exitFullscreen',
'mozCancelFullScreen',
'msExitFullscreen',
'webkitExitFullscreen'
'webkitExitFullscreen',
]
return names.reduce((prev, curr) =>
document[curr] ? curr : prev
)
return names.reduce((prev, curr) => (document[curr] ? curr : prev))
})()
2 changes: 1 addition & 1 deletion src/utils/format-time.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
export default function formatTime(current) {
let h = Math.floor(current / 3600)
let m = Math.floor((current - (h * 3600)) / 60)
let m = Math.floor((current - h * 3600) / 60)
let s = Math.floor(current % 60)

if (s < 10) {
Expand Down
6 changes: 2 additions & 4 deletions src/utils/fullscreen-change.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,7 @@ export default function fullscreenChange(type, callback) {
'fullscreenchange',
'mozfullscreenchange',
'MSFullscreenChange',
'webkitfullscreenchange'
'webkitfullscreenchange',
]
vendors.forEach(vendor =>
document[`${type}EventListener`](vendor, callback)
)
vendors.forEach(vendor => document[`${type}EventListener`](vendor, callback))
}
Loading

0 comments on commit f89327f

Please sign in to comment.