Skip to content

Comments

Implement reduced motion setting#211

Merged
danirabbit merged 2 commits intomainfrom
leolost/reduced-motion
Feb 20, 2026
Merged

Implement reduced motion setting#211
danirabbit merged 2 commits intomainfrom
leolost/reduced-motion

Conversation

@leolost2605
Copy link
Member

See https://gitlab.gnome.org/GNOME/xdg-desktop-portal-gnome/-/merge_requests/236 and flatpak/xdg-desktop-portal#1840

This PR also fixes read_all not including the other freedesktop appearance values, i.e. accent color and prefers color scheme.

@leolost2605 leolost2605 requested a review from a team November 8, 2025 18:09
@leolost2605 leolost2605 force-pushed the leolost/reduced-motion branch from 65bbebe to cf506d6 Compare November 8, 2025 18:11
@leolost2605 leolost2605 added this to OS 9 Nov 8, 2025
@leolost2605 leolost2605 moved this to Needs Review in OS 9 Nov 8, 2025
@lenemter
Copy link
Member

lenemter commented Nov 8, 2025

We don't need to add gsettings key for that. We should follow gnome's enable-animations key instead. I guess you tried to mimic what Gnome does there but gnome has separate options to reduce and to disable animation, so this doesn't affect us (at least not yet). But we difinitelly should look into this during OS 9 development cycle

image

@leolost2605
Copy link
Member Author

@lenemter I'm not quite sure what you're saying. Right now there is the gnome enable animations key that both we and gnome use to completely disable animations. This is useful for performance e.g. in a VM but was misused as an accessibility setting. Therefore recently gnome introduced along with the free desktop portal spec a new key for reduce animation i.e. to keep some animations or alter them so that they are not straining or irritating for more people. They added a new key to their gsettings (https://gitlab.gnome.org/GNOME/gsettings-desktop-schemas/-/merge_requests/114) I'm not sure is that the key they you meant we should use? if so I agree that's a possibility but I think since the implementation is private to GNOME we should probably use our own key instead of relying on theirs. If you mean we should continue using the old enable animations key I disagree since these are separate things (performance vs accessibility). This of course means we should switch the settings toggle for reduce motion to this new key and maybe migrate existing users.

@danirabbit
Copy link
Member

@lenemter can I get you to follow up here please? :)

Copy link
Member

@lenemter lenemter left a comment

Choose a reason for hiding this comment

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

My point was that this gsettings key is currently unused but I guess the UI part will be implemented in OS 9 so LGTM

@danirabbit danirabbit merged commit 147c91c into main Feb 20, 2026
3 of 4 checks passed
@danirabbit danirabbit deleted the leolost/reduced-motion branch February 20, 2026 17:23
@github-project-automation github-project-automation bot moved this from Needs Review to Done in OS 9 Feb 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants