Skip to content

feat(ozone/nexus): Assign VirtualKeyboard role to overlay windows OS-20973#129

Merged
caneraltinbasak merged 1 commit into28-x-y-bsfrom
cal/OS-20973
Apr 20, 2026
Merged

feat(ozone/nexus): Assign VirtualKeyboard role to overlay windows OS-20973#129
caneraltinbasak merged 1 commit into28-x-y-bsfrom
cal/OS-20973

Conversation

@caneraltinbasak
Copy link
Copy Markdown

Tooltip, popup, and menu windows were created without a Nexus surface
role (eNone). After the SetZOrderLevel changes gave the main window a
Graphics role, libbscompositor's role-based z-ordering placed the
role-less overlay windows behind the graphics surface.

Fix by assigning NEXUS_BrightSignSurfaceRole_eVirtualKeyboard to
tooltip, popup, and menu windows at creation time. In the compositor's
arrangement order (video > graphics > subtitles > ticker > virtual
keyboard > cursor), this ensures overlays appear above the main
graphics content.

@caneraltinbasak caneraltinbasak changed the title ozone/nexus: Assign VirtualKeyboard role to overlay windows OS-20973 feat(ozone/nexus): Assign VirtualKeyboard role to overlay windows OS-20973 Apr 13, 2026
@caneraltinbasak caneraltinbasak changed the base branch from rsmith/OS-20183 to 28-x-y-bs April 20, 2026 09:58
Tooltip, popup, and menu windows were created without a Nexus surface
role (eNone). After the SetZOrderLevel changes gave the main window a
Graphics role, libbscompositor's role-based z-ordering placed the
role-less overlay windows behind the graphics surface.

Fix by assigning NEXUS_BrightSignSurfaceRole_eVirtualKeyboard to
tooltip, popup, and menu windows at creation time. In the compositor's
arrangement order (video > graphics > subtitles > ticker > virtual
keyboard > cursor), this ensures overlays appear above the main
graphics content.
@caneraltinbasak caneraltinbasak merged commit 5165b7c into 28-x-y-bs Apr 20, 2026
3 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