From b5424b2f7df31df30ca9d40daa5167b4cb15d5c6 Mon Sep 17 00:00:00 2001 From: Mikael Finstad Date: Sun, 19 Mar 2017 15:50:43 +0100 Subject: [PATCH] Make end time the end of video by default --- src/renderer.jsx | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/renderer.jsx b/src/renderer.jsx index 6739e690..c886dc58 100644 --- a/src/renderer.jsx +++ b/src/renderer.jsx @@ -148,6 +148,11 @@ class App extends React.Component { } } + onDurationChange(duration) { + this.setState({ duration }); + if (!this.state.cutEndTime) this.setState({ cutEndTime: duration }); + } + setCutStart() { this.setState({ cutStartTime: this.state.currentTime }); } @@ -218,9 +223,6 @@ class App extends React.Component { const cutStartTime = this.state.cutStartTime; const cutEndTime = this.state.cutEndTime; const filePath = this.state.filePath; - if (cutStartTime === undefined || cutEndTime === undefined) { - return alert('Please select both start and time'); - } if (cutStartTime >= cutEndTime) { return alert('Start time must be before end time'); } @@ -264,7 +266,7 @@ class App extends React.Component { onRateChange={() => this.playbackRateChange()} onPlay={() => this.onPlay(true)} onPause={() => this.onPlay(false)} - onDurationChange={e => this.setState({ duration: e.target.duration })} + onDurationChange={e => this.onDurationChange(e.target.duration)} onTimeUpdate={e => this.setState({ currentTime: e.target.currentTime })} />