Skip to content

Commit 00f2832

Browse files
authored
Fix 87 initialising transcript media v2 (#114)
* QA docs for testing component interface * updated QA link * edited QA doc * mend * fix * refactored + QA * removed PORT in npm start * refactor fix * Removed some unused modules * transcript first not crash - but local storage still not working when component not initialised with media and transcript at same time * change mute icon * tested and working can add from url or from local media and it would restore session. Note that even if we know it's the same file, those are teo different sessions, as local media uses the file name, and url uses the full url to more uniquely identify transcriptions * done changes * Removed eslint ignore ref comment
1 parent c85e814 commit 00f2832

File tree

2 files changed

+216
-161
lines changed

2 files changed

+216
-161
lines changed

src/lib/TranscriptEditor/MediaPlayer/PlayerControls.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ import {
1616
faForward,
1717
faUndo,
1818
faVolumeUp,
19-
faVolumeOff
19+
faVolumeMute
2020
} from '@fortawesome/free-solid-svg-icons';
2121

2222
class PlayerControls extends React.Component {
@@ -117,7 +117,7 @@ class PlayerControls extends React.Component {
117117
<button
118118
value="Picture-in-picture"
119119
title="Picture-in-picture"
120-
className={ style.playerButton + " " + style.pip }
120+
className={ style.playerButton + ' ' + style.pip }
121121
onClick={ this.props.pictureInPicture }>
122122
<FontAwesomeIcon icon={ faTv } />
123123
</button>
@@ -127,7 +127,7 @@ class PlayerControls extends React.Component {
127127
title="Toggle Sound"
128128
className={ style.playerButton }
129129
onClick={ this.props.handleMuteVolume }>
130-
{ this.props.isMute ? <FontAwesomeIcon icon={ faVolumeOff } /> : <FontAwesomeIcon icon={ faVolumeUp } /> }
130+
{ this.props.isMute ? <FontAwesomeIcon icon={ faVolumeMute } /> : <FontAwesomeIcon icon={ faVolumeUp } /> }
131131
</button>
132132
</div>
133133
</div>

0 commit comments

Comments
 (0)