Skip to content

ExternalWindow: fix dialog parenting - #168

Merged
danirabbit merged 5 commits into
mainfrom
danirabbit/fixdialogparenting
Jan 30, 2026
Merged

ExternalWindow: fix dialog parenting#168
danirabbit merged 5 commits into
mainfrom
danirabbit/fixdialogparenting

Conversation

@danirabbit

Copy link
Copy Markdown
Member

Fixes dialog parenting in Wayland

Fixes dialog parenting in Wayland
@danirabbit
danirabbit requested a review from a team January 28, 2026 20:13
@danirabbit danirabbit moved this to Needs review in OS 8.1.1 Jan 28, 2026
@lenemter

Copy link
Copy Markdown
Member

@danirabbit How can I test this?

@danirabbit

danirabbit commented Jan 30, 2026

Copy link
Copy Markdown
Member Author

@lenemter try with open uri portal in ASHPD for example. In main it'll spawn in the center of the display. In this branch it'll be modal for ASHPD

@danirabbit

Copy link
Copy Markdown
Member Author

Doesn't fix exactly, but it's related to #164

@lenemter lenemter left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@danirabbit
danirabbit enabled auto-merge (squash) January 30, 2026 04:16
@danirabbit
danirabbit merged commit 3a83f72 into main Jan 30, 2026
4 checks passed
@danirabbit
danirabbit deleted the danirabbit/fixdialogparenting branch January 30, 2026 04:19
@github-project-automation github-project-automation Bot moved this from Needs review to Done in OS 8.1.1 Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants