Skip to content

Commit 56d69d8

Browse files
authored
Merge pull request #776 from Stremio/fix/player-subtitles-tracks-layout
Player: fix subtitles tracks layout
2 parents a8a8d65 + dbbf12c commit 56d69d8

File tree

2 files changed

+29
-10
lines changed

2 files changed

+29
-10
lines changed

src/routes/Player/SubtitlesMenu/SubtitlesMenu.js

+9-5
Original file line numberDiff line numberDiff line change
@@ -178,11 +178,15 @@ const SubtitlesMenu = React.memo((props) => {
178178
<div className={styles['variants-list']}>
179179
{subtitlesTracksForLanguage.map((track, index) => (
180180
<Button key={index} title={track.label} className={classnames(styles['variant-option'], { 'selected': props.selectedSubtitlesTrackId === track.id || props.selectedExtraSubtitlesTrackId === track.id })} data-id={track.id} data-origin={track.origin} data-embedded={track.embedded} onClick={subtitlesTrackOnClick}>
181-
<div className={styles['variant-label']}>
182-
{
183-
languages.label(!track.label.startsWith('http') ? track.label : track.lang)
184-
}
185-
<div className={styles['variant-origin']}>{t(track.origin)}</div>
181+
<div className={styles['info']}>
182+
<div className={styles['variant-label']}>
183+
{
184+
languages.label(!track.label.startsWith('http') ? track.label : track.lang)
185+
}
186+
</div>
187+
<div className={styles['variant-origin']}>
188+
{ t(track.origin) }
189+
</div>
186190
</div>
187191
{
188192
props.selectedSubtitlesTrackId === track.id || props.selectedExtraSubtitlesTrackId === track.id ?

src/routes/Player/SubtitlesMenu/styles.less

+20-5
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,6 @@
4242

4343
.language-label, .variant-label {
4444
flex: 1;
45-
max-height: 2.4em;
4645
font-size: 1.1rem;
4746
color: var(--primary-foreground-color);
4847
}
@@ -52,10 +51,6 @@
5251
text-overflow: ellipsis;
5352
}
5453

55-
.variant-label .variant-origin {
56-
color: var(--color-placeholder-text);
57-
}
58-
5954
.icon {
6055
flex: none;
6156
width: 0.5rem;
@@ -65,6 +60,26 @@
6560
background-color: var(--secondary-accent-color);
6661
}
6762
}
63+
64+
.variant-option {
65+
height: 4rem;
66+
67+
.info {
68+
flex: 1;
69+
display: flex;
70+
flex-direction: column;
71+
gap: 0.25rem;
72+
73+
.variant-label {
74+
line-height: 1.5rem;
75+
}
76+
77+
.variant-origin {
78+
font-size: 0.9rem;
79+
color: var(--color-placeholder-text);
80+
}
81+
}
82+
}
6883
}
6984
}
7085

0 commit comments

Comments
 (0)