Hide the cursor over the video once the bottom bar auto-hides #3

Merged

kyuno merged 2 commits from fix/player-cursor-hide into master

2026-09-15 22:52:34 +02:00

kyuno commented

2026-09-15 22:50:20 +02:00

Owner

Summary

When the player's bottom bar auto-collapses to just its seek line after a stretch of inactivity, the mouse cursor now hides too - toggled at the same two spots (scheduleControlsHide/showControls) that already toggle the bar's own .minimal state, so the two can't drift out of sync.

Test plan

npm run typecheck / npm run build on client/
Confirmed the compiled bundle carries both the classList calls and the new CSS rule
Not verified in an actual running player - no Emby login reachable from this sandbox

🤖 Generated with Claude Code

## Summary - When the player's bottom bar auto-collapses to just its seek line after a stretch of inactivity, the mouse cursor now hides too - toggled at the same two spots (`scheduleControlsHide`/`showControls`) that already toggle the bar's own `.minimal` state, so the two can't drift out of sync. ## Test plan - [x] `npm run typecheck` / `npm run build` on `client/` - [x] Confirmed the compiled bundle carries both the `classList` calls and the new CSS rule - [ ] Not verified in an actual running player - no Emby login reachable from this sandbox 🤖 Generated with [Claude Code](https://claude.com/claude-code)

kyuno added 1 commit 2026-09-15 22:50:21 +02:00

Hide the cursor over the video once the bottom bar auto-hides

Publier l'image Docker / test (pull_request) Failing after 15s

Details

Publier l'image Docker / publish (pull_request) Has been skipped

Details

26d32632c7

Asked for directly: the auto-hide behavior itself (bottom bar collapses
to just its seek line after a stretch of no activity) already shipped
earlier - a mouse cursor left sitting over the video afterward is the
same kind of clutter that was the point of hiding in the first place.

.playerWrap now gets a .cursorHidden class alongside #bottomBar's own
.minimal, toggled at the exact same two call sites
(scheduleControlsHide/showControls) so the two states can never drift
out of sync with each other. cursor: none on that class does the rest -
no new listener needed, since playerWrap's own pointermove/pointerdown
already call showControls() on any mouse activity, which is exactly
when the cursor should reappear too.

Verified: typecheck + build (client). Confirmed the compiled bundle
carries both the classList calls and the CSS rule. Not verified in an
actual running player - no Emby login reachable from this sandbox to
open a real video party and let the auto-hide timer elapse.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

kyuno added 1 commit 2026-09-15 22:52:23 +02:00

Merge branch 'master' into fix/player-cursor-hide

Publier l'image Docker / test (pull_request) Failing after 7s

Details

Publier l'image Docker / publish (pull_request) Has been skipped

Details

a01e0fa86f