Skip to content

Conversation

webfansplz
Copy link
Member

No description provided.

@webfansplz webfansplz requested a review from antfu August 14, 2025 15:35
@webfansplz webfansplz mentioned this pull request Aug 14, 2025
14 tasks
@CrabSAMA
Copy link
Contributor

Wow, you have the same idea as me! I found it have some performance when render large data in list and table, so I try to add virtual scroll library to resolve this problem.

But I try vue-virtual-scroll @tanstack/vue-virtual @vueuse/core -> useVirtualList , render list make it easy, but render virtual scroll native table have a little difficult. If use div list to simulate table, it will lost table's semantics.

So I have a question, should I introduce a UI component library to render tables?

@webfansplz
Copy link
Member Author

Wow, you have the same idea as me! I found it have some performance when render large data in list and table, so I try to add virtual scroll library to resolve this problem.

But I try vue-virtual-scroll @tanstack/vue-virtual @vueuse/core -> useVirtualList , render list make it easy, but render virtual scroll native table have a little difficult. If use div list to simulate table, it will lost table's semantics.

So I have a question, should I introduce a UI component library to render tables?

We do not recommend introducing additional component libraries in the project. I have temporarily skipped the virtual list optimization for the plugin detail table in this PR and plan to refactor the table into a flex layout to support it later.

@webfansplz webfansplz changed the title feat: add virtual list for data list feat: add virtual scroller for data list Aug 15, 2025
@CrabSAMA
Copy link
Contributor

We do not recommend introducing additional component libraries in the project. I have temporarily skipped the virtual list optimization for the plugin detail table in this PR and plan to refactor the table into a flex layout to support it later.

Well I got it, if need something help, plz tell me and I can try to refactor

@antfu antfu merged commit 37b93aa into main Aug 15, 2025
6 checks passed
@webfansplz webfansplz deleted the feat/virtual-list branch August 15, 2025 05:33
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.

3 participants