Compare commits
No commits in common. "e9e288c93c77c021fb2419e2b46e18011a9b9ce5" and "021f272a7dfdfe7e6ec4d4f3d63efa85e2b2be0d" have entirely different histories.
e9e288c93c
...
021f272a7d
3 changed files with 5 additions and 32 deletions
|
|
@ -386,9 +386,6 @@ Then put it behind your existing reverse proxy like everything else
|
||||||
was missing a grid-column span, so it (and not just the queue
|
was missing a grid-column span, so it (and not just the queue
|
||||||
popup above) was still squeezed and wrapping controls onto a
|
popup above) was still squeezed and wrapping controls onto a
|
||||||
second row — see docs/architecture.md
|
second row — see docs/architecture.md
|
||||||
- [x] Fixed an album's track rows collapsing to 3 lines on mobile —
|
|
||||||
there was always room for one, the breakpoint just assumed
|
|
||||||
otherwise — see docs/architecture.md
|
|
||||||
- [ ] Try a full party end-to-end with two real accounts (still the main gap)
|
- [ ] Try a full party end-to-end with two real accounts (still the main gap)
|
||||||
- [ ] Try casting to a real Chromecast (only the SDK's own bootstrap is
|
- [ ] Try casting to a real Chromecast (only the SDK's own bootstrap is
|
||||||
confirmed here — no real device/session)
|
confirmed here — no real device/session)
|
||||||
|
|
|
||||||
|
|
@ -42,3 +42,8 @@
|
||||||
|
|
||||||
.trackRow button.trackIconBtn:hover { border-color: var(--text); color: var(--text); background: none; }
|
.trackRow button.trackIconBtn:hover { border-color: var(--text); color: var(--text); background: none; }
|
||||||
.trackRow button.trackIconBtn svg { width: 12px; height: 12px; }
|
.trackRow button.trackIconBtn svg { width: 12px; height: 12px; }
|
||||||
|
|
||||||
|
@media (max-width: 600px) {
|
||||||
|
.trackRow { grid-template-columns: 24px 1fr; }
|
||||||
|
.trackRow .trackDuration, .trackRow button { grid-column: 2; justify-self: start; }
|
||||||
|
}
|
||||||
|
|
|
||||||
|
|
@ -4237,35 +4237,6 @@ which `align-items: center` legitimately varies by each one's own
|
||||||
content height) - confirming one shared row below, in the original
|
content height) - confirming one shared row below, in the original
|
||||||
track/transport/side left-to-right order.
|
track/transport/side left-to-right order.
|
||||||
|
|
||||||
## An album's track rows collapsed to 3 lines on mobile for no reason
|
|
||||||
|
|
||||||
Reported directly: there was room to fit an album's track rows
|
|
||||||
(index/name/duration/a small play-or-queue button) on one line on
|
|
||||||
mobile, but `trackList.css`'s own `@media (max-width: 600px)` forced
|
|
||||||
`.trackRow` down to a 2-column grid and pinned both `.trackDuration`
|
|
||||||
and the button to that same second column - auto-placement then
|
|
||||||
stacked the two of them into their own rows underneath the track name,
|
|
||||||
three rows total instead of one.
|
|
||||||
|
|
||||||
Removed outright rather than tuned - the numbers don't support it
|
|
||||||
having ever been needed. `.trackRow`'s fixed overhead (28px index +
|
|
||||||
2×12px gaps + `auto`-sized duration/button + `12px` horizontal padding)
|
|
||||||
comes to roughly 155-165px, and even the narrowest common phone width
|
|
||||||
(320px) leaves comfortably more than that for `.mainContent`'s own
|
|
||||||
side padding plus a `1fr` name column - `.trackName`'s existing
|
|
||||||
`text-overflow: ellipsis` was already doing exactly the job this
|
|
||||||
breakpoint assumed it couldn't.
|
|
||||||
|
|
||||||
Verified: full `typecheck`/`build` (client only). Checked directly, not
|
|
||||||
assumed: real compiled CSS, a handful of `.trackRow`s (short and
|
|
||||||
deliberately overlong names) loaded into two `<iframe>`s at 320px and
|
|
||||||
375px - this sandbox's own `resize_window` viewport emulation is
|
|
||||||
unreliable, see the mobile-nav/music-UI sections above. At both widths
|
|
||||||
every row's index/name/duration/button sat on one line (their vertical
|
|
||||||
ranges genuinely overlapping, not just similar by coincidence), left to
|
|
||||||
right in that order, with the long names confirmed actually truncated
|
|
||||||
(`scrollWidth > clientWidth`) rather than merely happening to fit.
|
|
||||||
|
|
||||||
## Known limitations
|
## Known limitations
|
||||||
|
|
||||||
- Playback always goes through `hls.js` against whatever Emby's
|
- Playback always goes through `hls.js` against whatever Emby's
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue