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

Svelte 5: Adding a delay param to svelte transitions causes the node to immediately appear for the duration of the delay #11270

Open
Mali-2 opened this issue Apr 21, 2024 · 3 comments
Milestone

Comments

@Mali-2
Copy link

Mali-2 commented Apr 21, 2024

Describe the bug

Adding a delay param to svelte transitions in svelte 5 causes the node to immediately appear for the duration of the delay. After this period the node disappears and starts its transition. Removing the delay param fixes this issue.

Reproduction

https://svelte-5-preview.vercel.app/#H4sIAAAAAAAAE6VQzUrEMBB-ldmodBciba8xLXjzAbxZD7Gd0GCalGSqlpB3l-winhTEywwz8_0xiWljMTLxlJhTCzLB7teVcUb7Wob4hpaQcRb9FsaykXEMZqV-cAOZZfWBIIFWE3LQducQR2URMujgF6gu_JqCctGQ8a66K0SLBHH279DBdSRFeNTKRjyVo5P1t4WTLxuRd-DdaM342qXjqeu_uIfSc1-qrC_AfnDpyugzIhcrObdgnCgJu5QmtGoX0DZNw2HagiqZLnPO_QNa6-FxxoAHWc9tEauNzj9p2v13SQ4fAqrbtrmp_q5-fuQ_Ig-Ocbb4yWiDExMUNszP-RPaLwNI8QEAAA==

Logs

No response

System Info

Browser: Chrome Version 123.0.6312.122 (Official Build) (arm64), m1 mac
Locally: "svelte": "5.0.0-next.110"

Severity

blocking an upgrade

@Xerios
Copy link

Xerios commented Apr 21, 2024

Quickly chiming in here as it was related to the #11216 fix

At first sight, changing fill: 'forwards' to fill: 'both' fixes this issue, although as stated in my comment it creates unexpected behavior but this time it's "stacked".

While testing delayed WAAPI animate() with fill: 'both' in my nested transition-heavy tree, clicking an expandable item multiple times would result in a stacked animation to be applied. Previous animation isn't aborted so it ends up as if we simply executed the call animate({delay:1000,...}) multiple times.

@svelte-kit-so-good
Copy link

Broken from alpha 82; working in alpha 81 (stackblitz demo).

@PatrickG
Copy link

Basically the same as #10876

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

No branches or pull requests

5 participants