Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Record Audio - Timeout and Visualizer #24457

Merged
merged 6 commits into from Aug 27, 2018
Merged

Record Audio - Timeout and Visualizer #24457

merged 6 commits into from Aug 27, 2018

Conversation

epeach
Copy link

@epeach epeach commented Aug 24, 2018

*Add a 30-second timeout to the record-audio feature.
*Add 'Recording' text and icon to make it clear when recording is happening. See below screenshots.

Before Recording:
screenshot from 2018-08-24 09-36-10

While Recording:
screenshot from 2018-08-24 09-36-20

@epeach epeach requested a review from Erin007 August 24, 2018 18:20
@@ -104,6 +122,12 @@ export default class AudioRecorder extends React.Component {
<div style={styles.buttonRow}>

<input type="text" placeholder="mysound1.mp3" onChange={this.onNameChange} value={this.state.audioName}/>
{this.state.recording &&
<span style={assetButtonStyles.button}>
<i style={{color: 'red', margin: 5}} className="fa fa-circle"/>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Especially since you already have style object in this file, let' remove the in-line styles here.

Copy link
Contributor

@Erin007 Erin007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM pending the merge conflict.

@epeach epeach merged commit bad7066 into staging Aug 27, 2018
@epeach epeach deleted the record-audio-timeout branch August 30, 2018 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants