-
-
Notifications
You must be signed in to change notification settings - Fork 610
chore: pin vue scroller to 2.0.0 beta 3 #3462
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
base: main
Are you sure you want to change the base?
Conversation
❌ Deploy Preview for elk-docs failed.
|
✅ Deploy Preview for elk-zone ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
weird 🤔, on pr preview the jump still present, maybe we need to pin virtual scroller to previous beta version 3 (only when virtual scrolling enabled => by default it is enabled); no idea why on my local working (maybe I need to dedupe dependencies on my local) |
|
I see. Then, let's try the beta 3 😅 I recently updated only masto.js version but I'm still not sure it's related to this issue. |
|
argggggg, it is even wrose, the issue maybe not in the scroller, this is weird, I will try to check it again later on my local deduping deps, no idea what's wrong (the deploy failed arggggg) 😒 |
@darrinlrogers can you test with the pr preview here
Looks like beta 5 has wrong logic: Akryum/vue-virtual-scroller#795
We should add vue scroller to ignoreDeps at renovate.json file until fixed
resolves #3457