fix(ui): playlist details overflow in spotify-based themes (#4184)
* test: ensure playlist details width * fix(test): simplify expectation for minWidth in NDPlaylistDetails Signed-off-by: Deluan <deluan@navidrome.org> * fix(test): test all themes Signed-off-by: Deluan <deluan@navidrome.org> --------- Signed-off-by: Deluan <deluan@navidrome.org>
This commit is contained in:
parent
bc733540f9
commit
7d1f5ddf06
3 changed files with 14 additions and 2 deletions
|
|
@ -259,7 +259,6 @@ export default {
|
||||||
},
|
},
|
||||||
details: {
|
details: {
|
||||||
fontSize: '.875rem',
|
fontSize: '.875rem',
|
||||||
minWidth: '75vw',
|
|
||||||
color: 'rgba(255,255,255, 0.8)',
|
color: 'rgba(255,255,255, 0.8)',
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
|
|
|
||||||
|
|
@ -204,7 +204,6 @@ export default {
|
||||||
},
|
},
|
||||||
details: {
|
details: {
|
||||||
fontSize: '.875rem',
|
fontSize: '.875rem',
|
||||||
minWidth: '75vw',
|
|
||||||
color: 'rgba(255,255,255, 0.8)',
|
color: 'rgba(255,255,255, 0.8)',
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
|
|
|
||||||
14
ui/src/themes/theme.test.js
Normal file
14
ui/src/themes/theme.test.js
Normal file
|
|
@ -0,0 +1,14 @@
|
||||||
|
import themes from './index'
|
||||||
|
import { describe, it, expect } from 'vitest'
|
||||||
|
|
||||||
|
describe('NDPlaylistDetails styles', () => {
|
||||||
|
const themeEntries = Object.entries(themes)
|
||||||
|
|
||||||
|
it.each(themeEntries)(
|
||||||
|
'%s should not set minWidth on details',
|
||||||
|
(themeName, theme) => {
|
||||||
|
const details = theme.overrides?.NDPlaylistDetails?.details
|
||||||
|
expect(details?.minWidth).toBeUndefined()
|
||||||
|
},
|
||||||
|
)
|
||||||
|
})
|
||||||
Loading…
Reference in a new issue