Skip to content

Commit 2c6a9dd

Browse files
author
Pietro Passarelli - News Labs
committed
UI Tweaks
1 parent ee20146 commit 2c6a9dd

File tree

7 files changed

+24
-59
lines changed

7 files changed

+24
-59
lines changed

src/index.js

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,7 @@ class App extends React.Component {
1919
isTextEditable: true,
2020
sttType: 'bbckaldi',
2121
analyticsEvents: [],
22-
title: 'Ted Talk Kate Kate Darling',
23-
fileName: 'Kate Darling Ted Talk'
22+
title: 'Ted Talk Kate Darling'
2423
};
2524
}
2625

@@ -198,13 +197,6 @@ class App extends React.Component {
198197
onChange={ e => this.handleChangeTranscriptTitle(e.target.value) }
199198
/>
200199
<br />
201-
<label>Transcript Name</label>
202-
<input
203-
type="text"
204-
onChange={ e => this.handleChangeTranscriptName(e.target.value) }
205-
value={ this.state.fileName }
206-
/>
207-
<br />
208200
<button onClick={ () => this.clearLocalStorage() }>Clear Local Storage</button>
209201
<hr/>
210202

src/lib/TranscriptEditor/MediaPlayer/PlayerControls.js

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -110,18 +110,6 @@ class PlayerControls extends React.Component {
110110
handleMuteVolume={ this.props.handleMuteVolume }
111111
/>
112112

113-
<div style={ {
114-
color: 'white',
115-
textOverflow: 'ellipsis',
116-
whiteSpace: 'nowrap',
117-
overflow: 'hidden',
118-
width: '20vw',
119-
display: 'inline-block',
120-
marginLeft: '1em'
121-
} }>
122-
{this.props.title}
123-
</div>
124-
125113
</div>
126114
);
127115
}

src/lib/TranscriptEditor/MediaPlayer/PlayerControls.module.css

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22

33
.playerControls {
44
/* margin: 1em; */
5-
margin-top: 0.5em;
5+
margin-top: 0.25em;
66
margin-left: 0.5em;
7-
margin: 1em;
7+
/* margin: 1em; */
88
/* display: flex; */
99
/* justify-content: flex-start; */
1010

@@ -28,11 +28,8 @@
2828
background: color-darkest-grey;
2929
font-size: 1em;
3030
cursor: pointer;
31-
/* border-color: black;
32-
border-width: 0.1rem;
33-
border-style: solid; */
34-
3531
margin-right: 0.3rem;
32+
margin-bottom: 0.3rem;
3633
}
3734

3835
.playBackRate{
@@ -72,6 +69,7 @@
7269
line-height: 48px;
7370
padding: 0 1em;
7471
background-color: color-darkest-grey;
72+
margin-bottom: 0.3rem;
7573
}
7674

7775
.currentTime {

src/lib/TranscriptEditor/MediaPlayer/index.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -373,11 +373,11 @@ class MediaPlayer extends React.Component {
373373

374374
const playerControlsSection = (
375375
<div className={ styles.controlsSection }>
376-
{/* <div className={ styles.titleBox }>
377-
<h1 className={ styles.title }>{ this.props.fileName? this.props.fileName : this.props.mediaUrl }</h1>
378-
</div> */}
376+
<div className={ styles.title }>
377+
{this.props.title? this.props.title: ''}
378+
</div>
379379
<PlayerControls
380-
title={ this.props.title? this.props.title: '' }
380+
// title={ this.props.title? this.props.title: '' }
381381
playMedia={ this.togglePlayMedia.bind(this) }
382382
isPlaying={ this.state.isPlaying }
383383
playbackRate={ this.state.playbackRate }

src/lib/TranscriptEditor/MediaPlayer/index.module.css

Lines changed: 10 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -4,33 +4,27 @@
44
background: black;
55
}
66

7+
.title{
8+
color: white;
9+
text-overflow: ellipsis;
10+
white-space: nowrap;
11+
overflow: hidden;
12+
width: 97vw;
13+
display: inline-block;
14+
margin-left: 0.5em;
15+
}
16+
717
.playerSection {
818
display: inline-flex;
919
align-items: flex-start;
1020
width: 100%;
1121
}
1222

13-
.controlsSection {
14-
/* width: 98%; */
15-
/* display: flex; */
16-
/* flex-direction: column; */
17-
}
18-
1923
.titleBox {
2024
border-bottom: 0.02em solid color-light-grey;
2125
margin-bottom: 1em;
2226
}
2327

24-
.title {
25-
margin: 1em;
26-
color: white;
27-
height: 1.3em;
28-
font-size: 16px;
29-
overflow: hidden;
30-
white-space: nowrap;
31-
text-overflow: ellipsis;
32-
}
33-
3428
.helpText {
3529
margin-top: 0px;
3630
margin-bottom: 0.1em;
@@ -41,8 +35,4 @@
4135
.hideInMobile {
4236
display: none;
4337
}
44-
45-
video {
46-
/* min-width: 50%; */
47-
}
4838
}

src/lib/TranscriptEditor/index.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -392,6 +392,7 @@ class TranscriptEditor extends React.Component {
392392
</section>
393393

394394
</div>
395+
395396
);
396397
}
397398
}

src/lib/TranscriptEditor/index.module.css

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -16,15 +16,16 @@
1616
flex-direction: column;
1717
justify-content: space-evenly;
1818
flex-wrap: nowrap;
19-
/* min-height: 100vh; */
2019
background-color: #f9f9f9;
2120
position: relative;
2221
}
2322

2423
.header {
2524
background-color: black;
2625
width: 100%;
27-
/* height: 2em; */
26+
}
27+
.aside{
28+
margin-left:0.5em;
2829
}
2930

3031
.nav {
@@ -74,7 +75,6 @@
7475
.header {
7576
background-color: black;
7677
width: 100%;
77-
/* height: 2em; */
7878
}
7979

8080
.nav {
@@ -87,7 +87,6 @@
8787
}
8888

8989
.row {
90-
/* display: flex; */
9190
margin-top: 1em;
9291
}
9392

@@ -124,18 +123,16 @@
124123
.cog {
125124
font-weight: lighter;
126125
}
127-
128126
}
129127

130128
.help {
131129
cursor: pointer;
132130
float: right;
133131
padding-right: 0.5em;
134-
padding-left: 11em;
132+
/* padding-left: 11em; */
135133
margin: 0.5em 0;
136134
color: white;
137135
display: block;
138-
/* font-size: 0.8em; */
139136
font-weight: lighter;
140137
margin-top: 0.5em;
141138
margin-right: 6em;
@@ -148,7 +145,6 @@
148145

149146
.helpMessage span {
150147
display: block;
151-
/* font-size: 0.8em; */
152148
font-weight: lighter;
153149
margin-top: 1em;
154150
}

0 commit comments

Comments
 (0)