Skip to content

fix(desktop): keep the HUD from demoting the app to accessory - #186

Merged
kamrul1157024 merged 1 commit into
kamrul1157024:mainfrom
kmtusher97:fix/hud-accessory-demote
Sep 7, 2026
Merged

fix(desktop): keep the HUD from demoting the app to accessory#186
kamrul1157024 merged 1 commit into
kamrul1157024:mainfrom
kmtusher97:fix/hud-accessory-demote

Conversation

@kmtusher97

Copy link
Copy Markdown
Collaborator

Summary

Follow-up to #184. On macOS, showing the approval HUD while another app was in fullscreen kicked you out of fullscreen, and the Helios Dock icon lost its running indicator and became unclickable.

The cause is setVisibleOnAllWorkspaces(true, …) in hud.ts. By default Electron reacts to that call by transforming the app's process type between regular and accessory (UIElement). An accessory app has no Dock indicator and an unresponsive Dock icon — and the activation change is what drags the frontmost app out of its fullscreen space.

Passing skipTransformProcessType: true keeps the HUD visible on every space (including over fullscreen apps) without restyling the process.

This is independent of, and pre-dates, the anchor-pin work in #184 — that PR only touched positioning; this touches the panel's space/activation behavior.

Test plan

  • npm run typecheck — clean
  • npm test — 297 pass, 0 fail
  • Verified live in the dev build: with an app in macOS fullscreen, the HUD now overlays without exiting fullscreen, and the Dock icon keeps its indicator and stays clickable.

setVisibleOnAllWorkspaces defaults to transforming the process type
between regular and accessory when the panel joins all spaces. That
flip drops the Dock indicator, makes the Dock icon unclickable, and
drags the frontmost app out of its fullscreen space. Pass
skipTransformProcessType so the HUD stays visible everywhere without
restyling the process.
@kamrul1157024
kamrul1157024 merged commit 9f30f5f into kamrul1157024:main Sep 7, 2026
6 checks passed
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.

2 participants