mobile icon indicating copy to clipboard operation
mobile copied to clipboard

Add piece animation speed setting

Open EmmetSchuler opened this issue 1 year ago • 6 comments

Although the app already has a toggle for animating the piece movements, there is not yet an option to change how fast or slow they move. This functionality does appear on the website.

Screenshot_20240609-102731.png

EmmetSchuler avatar Jun 09 '24 14:06 EmmetSchuler

Is it really useful to have different speed?

veloce avatar Jun 09 '24 17:06 veloce

In my opinion, definitely! When people tap to move pieces, especially in faster time controls they'd like it to respond quicker. And some people also prefer slower and more calm movements when playing in general. I think this is pretty important for the overall enjoyment of many different types of players.

EmmetSchuler avatar Jun 09 '24 17:06 EmmetSchuler

It might also be good to set default animation speeds for each time control. Seems like on the website default values for bullet are instant movement, 3+0 is fast movement and 5+3 is around regular movement. In all honesty I don't really understand how the movement speed setting works. From what I've tested it seems to always be instant movement in bullet, even when I set the animation speed to "slow". Maybe specific time controls force a different speed, and only in classical, rapid, and slower blitz can you change the speeds?

EmmetSchuler avatar Jun 09 '24 17:06 EmmetSchuler

I didn't know that instant movement was the default in bullet.

Maybe specific time controls force a different speed, and only in classical, rapid, and slower blitz can you change the speeds?

I think you're right. It has to be fixed in bullet, otherwise there would be an option per time control, and we don't want to go that far.

veloce avatar Jun 10 '24 12:06 veloce

Actually I checked the server code to understand, and it's computed like that:

https://github.com/lichess-org/lila/blob/9a826e85e7fb67fdb76c464b9ed5f04dac1e60f6/modules/round/src/main/JsonView.scala#L337-L341

veloce avatar Jun 10 '24 12:06 veloce

Don't forget about puzzle storm, as well. Seems like that defaults to instant move speed too

EmmetSchuler avatar Jun 10 '24 22:06 EmmetSchuler