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

Make filter icon animation smoother #1290

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Riksu9000
Copy link
Contributor

With these changes, the animation won't stop abruptly.

Previously clicking on the filter multiple times or clicking on another filter while the animation is playing can cause the animation to end abruptly. Now the animation always plays to the end before it can start again.

The animation length is 700ms, so the timeout was made shorter.

src/Layouts/FilterPaneRow.vala Outdated Show resolved Hide resolved
@Riksu9000 Riksu9000 marked this pull request as draft May 12, 2024 18:00
@Riksu9000
Copy link
Contributor Author

I noticed an issue where the selected style is not always applied. I'll take another look tomorrow.

With these changes, the animation won't stop abruptly.

Previously clicking on the filter multiple times or clicking on another
filter while the animation is playing can cause the animation to end
abruptly. Now the animation always plays to the end before it can start
again.

The animation length is 700ms, so the timeout was made shorter.
@Riksu9000 Riksu9000 marked this pull request as ready for review May 12, 2024 18:08
@Riksu9000
Copy link
Contributor Author

Never mind, it was pretty simple.

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.

None yet

2 participants