Skip to content

Hide cursor during background reveal in song select#36252

Merged
peppy merged 3 commits intoppy:masterfrom
UltraDrakon:hide-cursor-during-background-reveal
Jan 13, 2026
Merged

Hide cursor during background reveal in song select#36252
peppy merged 3 commits intoppy:masterfrom
UltraDrakon:hide-cursor-during-background-reveal

Conversation

@UltraDrakon
Copy link
Copy Markdown
Contributor

Addresses #36238

When using the background reveal feature, the cursor was intrusive and blocked the view of the beatmap background.

This PR makes the cursor fade out in sync with the background reveal animation. The behavior is:

  • Cursor fades out when the background reveal starts
  • Moving the mouse while viewing the background shows the cursor immediately
  • After 1 second of no mouse movement, the cursor fades out again
  • Cursor returns to normal when releasing the mouse button

Note: Also includes a small fix to MenuCursorContainer to preserve cursor rotation state during hide/show cycles. I am not entirely sure if this was needed but I decided to include it to keep cursor rotation while revealing the background.

- Implement IProvideCursor in SongSelect to hide cursor when background is revealed
- Cursor reappears on mouse movement and hides again after 1 second of inactivity
- Fix MenuCursorContainer to preserve drag rotation state during hide/show cycles
@nagi-desuuu
Copy link
Copy Markdown

Supersedes #36248

@peppy peppy self-requested a review January 9, 2026 08:56
@peppy peppy enabled auto-merge January 9, 2026 09:02
@peppy peppy disabled auto-merge January 13, 2026 00:41
@peppy peppy merged commit eb14176 into ppy:master Jan 13, 2026
6 of 9 checks passed
@UltraDrakon UltraDrakon deleted the hide-cursor-during-background-reveal branch January 14, 2026 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants