Skip to content

Feature/hide fullscreen#4514

Open
tiago-coderia wants to merge 2 commits into
ramensoftware:mainfrom
tiago-coderia:feature/hide-fullscreen
Open

Feature/hide fullscreen#4514
tiago-coderia wants to merge 2 commits into
ramensoftware:mainfrom
tiago-coderia:feature/hide-fullscreen

Conversation

@tiago-coderia

Copy link
Copy Markdown

Changelog

If this pull request updates an existing mod, describe the changes below:

This PR improves fullscreen behavior for Taskbar Music Lounge.

Previously, the widget could remain visible above fullscreen applications and videos because it was created as a topmost window in the ZBID_IMMERSIVE_NOTIFICATION band.

Changes made:

  • Added explicit fullscreen detection based on the foreground window size.
  • Replaced QUERY_USER_NOTIFICATION_STATE checks with the new fullscreen detection.
  • Removed WS_EX_TOPMOST.
  • Replaced HWND_TOPMOST with HWND_NOTOPMOST.
  • Changed the window band from ZBID_IMMERSIVE_NOTIFICATION to ZBID_DEFAULT.

As a result, the widget now automatically hides when applications enter fullscreen mode (games, YouTube, Netflix, etc.) and reappears when returning to normal desktop usage.

Changelog

Improve fullscreen detection and hiding behavior.
Remove topmost window behavior to prevent overlaying fullscreen applications.
Use the default Z-band to better integrate with desktop and taskbar visibility.

  • Improve fullscreen detection and hiding behavior.
  • Remove topmost window behavior to prevent overlaying fullscreen applications.
  • Use the default Z-band to better integrate with desktop and taskbar visibility.

Mod authorship

If this pull request introduces a new mod, please complete the section below.

This mod was created by:

    • The submitter, without AI assistance
    • The submitter, with AI assistance
    • Claude
    • ChatGPT
    • Gemini
    • Another AI (please specify):
    • Other (please specify):

Please select the options that best apply. Your selection does not affect the acceptance criteria, but it helps reviewers understand the context of the code and provide relevant feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant