Skip to content

Conversation

@Sriram-L
Copy link

@Sriram-L Sriram-L commented Jan 22, 2026

Fixes #2534

Fix description:
The fix basically fetches current value of "Remove animations" in Android, and the board settings UI refreshes so that after fetching the value from system settings, the toggle option will be disabled if the system animation is disabled, and enabled when system animation is enabled.

Attaching a recording of how the fix works.

piece_animation_setting.mp4

system setting information and then updating the UI based on the value
of the setting, by letting the user to be able to toggle piece
animations only when system animations are enabled, otherwise do not
allow to change the setting.
@tom-anders
Copy link
Collaborator

tom-anders commented Jan 22, 2026

Thanks for the contribution!

However, I think this does not fix the issue in #2534. The reporter of this issue suggested that the piece animation toggle should work, regardless of whether animations are disabled on the system level or not.

@Sriram-L
Copy link
Author

Thanks for the contribution!

However, I think this does not fix the issue in #2534. The reporter of this issue suggested that the piece animation toggle should work, regardless of whether animations are disabled on the system level or not.

Thanks for the reply!

But I checked when "Remove animations" is enabled, piece animations do not work no matter what. So how can we satisfy this requirement regardless of this setting?

@tom-anders
Copy link
Collaborator

Thanks for the contribution!
However, I think this does not fix the issue in #2534. The reporter of this issue suggested that the piece animation toggle should work, regardless of whether animations are disabled on the system level or not.

Thanks for the reply!

But I checked when "Remove animations" is enabled, piece animations do not work no matter what. So how can we satisfy this requirement regardless of this setting?

No idea - you'd have to do some research if flutter somehow allows overriding/ignoring the system's "remove animations" setting.

@veloce
Copy link
Contributor

veloce commented Jan 26, 2026

Do we really want to follow the android system "Remove animations" toggle for our piece animations? I don't think this is a good idea, it looks like 2 very different things to me.

@tom-anders
Copy link
Collaborator

tom-anders commented Jan 26, 2026

Do we really want to follow the android system "Remove animations" toggle for our piece animations? I don't think this is a good idea, it looks like 2 very different things to me.

Yes, the problem is that currently we do follow this toggle in the android system (see the original issue): When "remove animations" is active in the Android systems, them piece animations currently are disabled as well. I assume that flutter somehow internally notices this and overrides the animation duration. The OP in the original issue argues that piece animations should ignore that setting. So we'd need to find some way to stop flutter respecting that system toggle.

@veloce
Copy link
Contributor

veloce commented Jan 27, 2026

I had no idea that Flutter did this. But it makes sense in a way, this can be used to save battery for instance.

Not sure we can do sth about it, it needs an investigation.

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.

3 participants