Realank / flutter_section_table_view

A iOS like table view including section, row, section header and divider
MIT License
73 stars 23 forks source link

Issue with the scrolling performance #5

Open numen31337 opened 5 years ago

numen31337 commented 5 years ago

In case of using animateTo or even jumpTo function to scroll the table to some target area, every single cell on the way will be rendered. This feature is completely unusable when using for jumping to some area of a long list (500 items and more) as it will render every single cell on the way to the target. I would expect at least jumpTo function to avoid rendering every single cell on the way.

Realank commented 5 years ago

Thank you for submitting this great issue, it's a problem, but the render things are handled by flutter framework itself, I just wrapped ListView widget, jumpTo method is also ListView's, so I have no idea to improve the performance of this package currently, do you have any ideas about this? thanks

numen31337 commented 5 years ago

I‘m a new to the Flutter ecosystem as we all are :) Unfortunately it's hard for me to purpose some solution right away. I will look into this issue a bit later and purpose something if I will come up with some ideas.

jizhigang commented 5 years ago

If I want to use the refresh function, do I need to determine the height of each cell?

numen31337 commented 5 years ago

Here is the relevant discussion, just for reference https://github.com/flutter/flutter/issues/12319

jizhigang commented 5 years ago

thank you 😄

------------------ 原始邮件 ------------------ 发件人: "Oleksandr"notifications@github.com; 发送时间: 2019年4月15日(星期一) 晚上6:25 收件人: "Realank/flutter_section_table_view"flutter_section_table_view@noreply.github.com; 抄送: "!!"1196581160@qq.com; "Comment"comment@noreply.github.com; 主题: Re: [Realank/flutter_section_table_view] Issue with the scrollingperformance (#5)

Here is the relevant discussion, just for reference flutter/flutter#12319

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

Realank commented 5 years ago

@jizhigang no you shouldn't. height is necessary for calculating scroll offset related callbacks