-
-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Map animation is a touch too fast #227
Comments
@Mrnofish sure i ll make it configurable! |
@Mrnofish it is now implemented |
@cb8121 really sorry. Will fix it |
FWIW it's working for me. There is some room for small improvements (e.g. the slider setting a delay value instead of a speed? thus apparently working the opposite that one would expect) but it's fine otherwise, thanks for taking care of it. (I'm leaving this open since it's being used to track the regression.) |
@Mrnofish indeed the animation speed is defined by a delay. Would it be better to have a speed like with default to 1 and the going slower (0.5) or faster (1.5) ? |
Since the proposed scale would be completely arbitrary (real time not being a useful option), it would come down to the tuning. What I was pointing out, is that a slider labeled "speed" intuitively associates bigger numbers to higher speeds, whereas now it works in reverse (in fact sliding to the right, which is generally associated with higher/louder/faster/etc, causes the animation to slow down instead.) I can certainly live with the way it is, though, and simply renaming the slider so that it's consistent with its function would probably be okay for most. |
@Mrnofish yes i agree it is not ok as it is. I will change it to some arbitrary "speed factor" (1 being the default) |
@cb8121 I am.sorry I can't see what s happening. Was it working before ? |
Yes, it was working prior to v1.8.1 |
Interesting! will try to play store build and report to you |
The map animation is a touch fast for my tastes, it makes interpreting the movement of the rain cells harder than it needs be. Would it be possible to perhaps add some way to customize the speed?
The text was updated successfully, but these errors were encountered: