Skip to content

Replace org.freedesktop.ScreenSaver with org.freedesktop.portal.Inhibit#118

Merged
diegotori merged 5 commits intofluttercommunity:mainfrom
shinnkka:main
Apr 4, 2026
Merged

Replace org.freedesktop.ScreenSaver with org.freedesktop.portal.Inhibit#118
diegotori merged 5 commits intofluttercommunity:mainfrom
shinnkka:main

Conversation

@shinnkka
Copy link
Copy Markdown
Contributor

Description

org.freedesktop.portal.Inhibit is part of the XDG Desktop Portal API, which provides the modern, cross-desktop mechanism for handling session state changes. The Portal documentation explicitly targets sandboxed and Wayland applications, reflecting the current desktop architecture. In contrast, org.freedesktop.ScreenSaver was originally designed for basic idle inhibition and exists primarily for backwards compatibility, lacking support for newer session controls.
This PR enhances sandbox support on Linux desktops and improves integration with modern desktop environments.

doc:https://flatpak.github.io/xdg-desktop-portal/docs/doc-org.freedesktop.portal.Inhibit

@diegotori
Copy link
Copy Markdown
Collaborator

@c-xk Thanks for your proposed changes. Can you add unit tests covering your potential changes? Thanks in advance.

@shinnkka
Copy link
Copy Markdown
Contributor Author

I can pass those unit tests on my develop platform

@diegotori
Copy link
Copy Markdown
Collaborator

@shinnkka looks like you'll have to re-sync your changes. That and because of the new Github Actions workflow added recently, you'll have to modify the linux_integration_test job so that it's able to run the integration tests on that platform.

Thanks in advance for your contributions thus far.

@shinnkka shinnkka requested a review from diegotori April 4, 2026 03:39
Copy link
Copy Markdown
Collaborator

@diegotori diegotori left a comment

Choose a reason for hiding this comment

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

LGTM. I'll release it on Monday. Thanks for your contribution.

@diegotori diegotori merged commit 961cd34 into fluttercommunity:main Apr 4, 2026
191 of 194 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