tables icon indicating copy to clipboard operation
tables copied to clipboard

Enhancement/279 pagination for tables ui

Open grnd-alt opened this issue 2 months ago • 3 comments

Design

image

Performance

Loading Time for 3k Rows with pagination

https://github.com/nextcloud/tables/assets/56235737/740f9864-50f6-45f6-9a94-fb48ac33d7fa

3k Rows Without pagination

https://github.com/nextcloud/tables/assets/56235737/f9b98ff4-0f25-4ba5-901e-3416a5d8f2ef

grnd-alt avatar Apr 10 '24 14:04 grnd-alt

I realize this is still a draft, but just wondering if this PR adds pagination or virtual scrolling? There seems to be a mismatch between the title and the commit message :sweat_smile:

enjeck avatar Apr 15 '24 02:04 enjeck

I realize this is still a draft, but just wondering if this PR adds pagination or virtual scrolling? There seems to be a mismatch between the title and the commit message 😅

Yeah it's because the initial issue planned to implement v-scrolling but that's harder to do on html tables than expected hence we decided to look into pagination

grnd-alt avatar Apr 15 '24 08:04 grnd-alt

@jancborchardt Updated With New Screenshot

grnd-alt avatar Apr 29 '24 07:04 grnd-alt

@jancborchardt Updated With New Screenshot

It doesn’t look like it has the drop shadow, but that looks fine to me, as long as the background is opaque color-main-background – so the element is still readable when there is a long table below it. :)

jancborchardt avatar May 02 '24 09:05 jancborchardt