Skip to content
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

Animation non block #1347

Merged
merged 3 commits into from
Aug 12, 2022
Merged

Animation non block #1347

merged 3 commits into from
Aug 12, 2022

Conversation

Ralim
Copy link
Owner

@Ralim Ralim commented Aug 3, 2022

Buttons can now skip an animation in progress
Animations are faster

  • Please check if the PR fulfills these requirements
  • The changes have been tested locally
  • There are no breaking changes

@River-Mochi
Copy link
Contributor

This is related to Issue ticket #1330 also.

@discip
Copy link
Collaborator

discip commented Aug 3, 2022

@Ralim
I tried that with my trusty TS80P and have to say:
Only if you press and hold B the animations are skipped.
If you however only press B quickly in a consecutive manner there is not a whole lot speed change observable imho.

@discip
Copy link
Collaborator

discip commented Aug 10, 2022

@Ralim
What was this PR supposed to achieve?
Make the animation faster, leave the animation out completely on consecutive button-press?
How can I assist?

@Ralim
Copy link
Owner Author

Ralim commented Aug 11, 2022

Slightly faster animation
Dont allow animation playing to block you moving to the next item; so holding button or pressing rapidly should cancel animation and move on to next item as fast as possible.

@River-Mochi
Copy link
Contributor

some people said that on pinecil, they did not like the new animations b/c it was not keeping up with button presses. They want the ability to press the buttons fast and not be delayed waiting for the entire animation to complete.
They were used to this in past firmware, but with the new animations, it affected their QOL and user experience.

@Ralim Ralim merged commit 7fcd2b7 into dev Aug 12, 2022
@Ralim Ralim deleted the animation-non-block branch August 12, 2022 00:36
@discip
Copy link
Collaborator

discip commented Aug 12, 2022

@Ralim

or pressing rapidly should cancel animation and move on to next item as fast as possible.

Yeah, that is what I expected when I first read the description of this PR.
But as I pointed out above animations are only skipped when B is held down, there is little to no change if B is pressed in rapid succession, though!
On the Pinecil it seems to work a bit faster compared to TS80P, since it has a faster mpu, but it does not keep up on either of them!

ps: Does it work for you the way you described it?

@River-Mochi
Copy link
Contributor

River-Mochi commented Aug 12, 2022

I tried the beta and I like it , it's much faster now when I press the button fast, it seems to keep up on the display. I had no issue just pressing the button slower before this beta ( I don't want to break pinecil so I'm gentle on buttons) , but other people in discord did complain about it not being like older firmware. therefore Ralim made the beta. so you could skip animations the moment you press the button.

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.

Menu animation prevent moving quickly
3 participants