Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Access/Dialog: Center and keep above on wayland #114

Merged
merged 6 commits into from
Aug 15, 2024

Conversation

leolost2605
Copy link
Member

Fixes #115

Requires elementary/gala#2027 (Otherwise it will crash when closed)

@leolost2605 leolost2605 requested a review from a team August 14, 2024 20:48
@danirabbit
Copy link
Member

Ah I forgot this uses request portal! That's great because that makes sense for all of those tbh.

I'll have to write packaging for the pantheon lib so marking this as 'blocked' 😅

Copy link
Member

@danirabbit danirabbit left a comment

Choose a reason for hiding this comment

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

Ah okay so looking at this more, these request dialogs usually have a parent. I'm guessing that in this case we don't get a parent because it's being called from Gala and not really from system settings. So we probably should make sure we only do center/on top if we don't have a parent window

@leolost2605
Copy link
Member Author

Ah right I forgot about that, fixed it :)

@danirabbit

This comment was marked as resolved.

@danirabbit

This comment was marked as resolved.

Copy link
Member

@danirabbit danirabbit left a comment

Choose a reason for hiding this comment

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

This is great! Thank you :)

@danirabbit danirabbit enabled auto-merge (squash) August 15, 2024 23:26
@danirabbit danirabbit merged commit 1a825f6 into main Aug 15, 2024
2 of 4 checks passed
@danirabbit danirabbit deleted the leolost/center-access-dialog branch August 15, 2024 23:26
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.

Center and Keep on Top display settings dialog
2 participants