Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Crash when loading 10000 messages #82

Open
sorunokoe opened this issue Nov 4, 2024 · 2 comments
Open

Crash when loading 10000 messages #82

sorunokoe opened this issue Nov 4, 2024 · 2 comments
Assignees
@ekazaev
Labels
enhancement New feature or request

Comments

@sorunokoe
Copy link

Screenshot 2024-11-04 at 11 09 04
@ekazaev
Copy link
Owner

ekazaev commented Nov 18, 2024

@sorunokoe A little bit more details please. How exactly you achieved it?

@ekazaev ekazaev self-assigned this Nov 18, 2024
@ekazaev
Copy link
Owner

ekazaev commented Nov 18, 2024 •
edited
Loading

@sorunokoe I managed to reproduce it. I generated 10000 messages initially. It seems that you can easily open this bug for apple itself :). It is an internal crash in UICollectionView and it is not reproducible in the IOS below 18. And IOS 18 itself is famously buggy. Seems they are keep breaking UIKit in favor of SwiftUI. So there is nothing I can do about it. Ill monitor it though. If ill find a better solution - I will obviously update it. But ChatLayout itself doesn't do anything forbidden.

Solution: You will have to use pagination if you are relying on this amount of cells in the chat. It is a good idea in general.

@ekazaev ekazaev added the enhancement New feature or request label Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ekazaev ekazaev

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ekazaev @sorunokoe