Skip to content
This repository has been archived by the owner on Nov 5, 2023. It is now read-only.

fix: sharing nodes states (#227) #230

Merged
merged 1 commit into from
Jun 4, 2022
Merged

fix: sharing nodes states (#227) #230

merged 1 commit into from
Jun 4, 2022

Conversation

zumm
Copy link
Contributor

@zumm zumm commented Jun 3, 2022

This fix uses internalItem.index as keys and timestamp of last pageProvider change as prefix of keys, Simplest solution. I don't want to change internal item type, so it is without key caching sadly.

@netlify
Copy link

netlify bot commented Jun 3, 2022

👷 Deploy request for vue-virtual-scroll-grid accepted.

Name Link
🔨 Latest commit 1edf58c
🔍 Latest deploy log https://app.netlify.com/sites/vue-virtual-scroll-grid/deploys/629a9e274fb971000862d9e3

Copy link
Owner

@rocwang rocwang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@rocwang rocwang merged commit a13e411 into rocwang:main Jun 4, 2022
@github-actions
Copy link

github-actions bot commented Jun 4, 2022

🎉 This PR is included in version 1.6.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants