Skip to content

Conversation

@danirabbit
Copy link
Member

@danirabbit danirabbit commented Nov 6, 2025

Make sure we don't show System Settings desktop file in other desktop environments and that we bail out if someone tries to run it manually. Prevents running a system settings that won't work as expected when folks have multiple desktop environments installed

@danirabbit danirabbit requested a review from a team November 6, 2025 21:55
@danirabbit danirabbit moved this from In progress to Needs review in Distro-agnostic Pantheon Nov 6, 2025
@danirabbit danirabbit requested a review from ryonakano November 7, 2025 22:18
Copy link
Member

@ryonakano ryonakano left a comment

Choose a reason for hiding this comment

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

Confirmed that it now works as expected:

user@fedora:~/work/tmp/switchboard (danirabbit/only-pantheon =)$ io.elementary.settings

(process:3341): switchboard-CRITICAL **: 09:02:51.263: Application.vala:265: System Settings can only be run in a Pantheon desktop session
[ble: exit 1]
user@fedora:~/work/tmp/switchboard (danirabbit/only-pantheon =)$

Also the app icon is not visible in Activities Overview even if I type to search. Thanks!

@github-project-automation github-project-automation bot moved this from Needs review to Reviewer approved in Distro-agnostic Pantheon Nov 8, 2025
@ryonakano ryonakano merged commit 859cb3d into main Nov 8, 2025
4 checks passed
@ryonakano ryonakano deleted the danirabbit/only-pantheon branch November 8, 2025 00:06
@github-project-automation github-project-automation bot moved this from Reviewer approved to Done in Distro-agnostic Pantheon Nov 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants