-
Notifications
You must be signed in to change notification settings - Fork 186
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
fix(Gallery): remove add slide change animation to queue at not looped #8292
fix(Gallery): remove add slide change animation to queue at not looped #8292
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
size-limit report 📦
|
e2e tests |
👀 Docs deployed
Commit 653f453 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #8292 +/- ##
==========================================
- Coverage 95.54% 95.54% -0.01%
==========================================
Files 404 404
Lines 11641 11637 -4
Branches 3859 3859
==========================================
- Hits 11122 11118 -4
Misses 519 519
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
✅ v7.1.3 🎉 |
Описание
В #8042 реализации зацикленной и незацикленной галерии были объединены. При объединении была изменена логика перелистывания слайдов при изменении slideIndex. В новой реализации перелистывание происходило с помощью очереди анимаций. Поэтому при быстром изменении
slideIndex
создается не очень хорошее ощущение, что слайды перелистываются заторможено. Поэтому нужно для незацикленной галереи сделать перелистывание без использования очереди анимацийИзменения
Убрал анимацию через js и добавление ее в очередь при перелистывании слайдов в незацикленной галерее.
Release notes
Исправления
slideIndex