-
-
Notifications
You must be signed in to change notification settings - Fork 7k
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
[Bug Report] Data Table render performance #9946
Comments
Issue seems to be more related to Vuetify base components than the Data Grid itself. Another codepen here https://codepen.io/rogeralsing/pen/GRgZzZz This renders the same 100 rows with buttons, but in a simple div. Same with other components such as the Vuetify text field. Replacing the content with standard html buttons makes the render speed as fast as one would expect. |
It just has way too many props, as well as checking if it's inside a |
Is there any update or workaround ? i have same issue unfortunately . |
Hi @johnleider , i guess this bug related to vuejs/vue#9371 issue . |
This is about 50% faster now in v3. It still takes around a second with 4x cpu slowdown so if you're targeting mobile devices you should try to avoid rendering thousands of components at once, consider using a recycler list instead. |
@KaelWD What do you mean with recycler list? Something like https://vuetifyjs.com/en/components/virtual-scroller/ ? With this component we would have to build pagination and other things needed from data-table isn't it? I'm looking forward to the new v3 data-table 😃 |
Environment
Vuetify Version: 2.1.14
Vue Version: 2.6.9
Browsers: Chrome 78.0.3904.108
OS: Mac OS 10.15.1
Steps to reproduce
Click the green button on the Code pen example.
This renders 100 items in the Vuetify Data Table.
Expected Behavior
100 items with buttons and icons is an artificial use case, but the long render time hints that something is not as expected.
I would expect 100 items to render within milliseconds.
Actual Behavior
Render time is within 0.5 - 1 seconds
Reproduction Link
https://codepen.io/rogeralsing/pen/MWYyVXz
The text was updated successfully, but these errors were encountered: