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.

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

UI is blocking when there are many messages in the conversation #34

Open
ChrisLuck opened this issue Dec 4, 2015 · 5 comments
Open

Comments

@ChrisLuck
Copy link

Hi, I noticed that the UI is blocked when the app is receiving messages. On my Nexus4 this usually takes about 5 second per received message. On one hand, this ui blocking is good because i learned how to 'use' it as a progress indicator. Whenever i get the app in the foreground i am trying to scroll. When the scrolling works i know there are no messages. But when the scrolling is blocked i know that i have to wait some seconds to read my new messages. Maybe you should consider inegrating some kind of progress indicator for receive/decrypt and make the UI non-blocking.

@janimo
Copy link
Owner

janimo commented Dec 4, 2015

I agree this is needed, and it is planned for attachments. I did not think it would be noticeable for plain messages. Does it really take 5 seconds per received text message? If so it may be an unrelated bug.

@ChrisLuck
Copy link
Author

I just received a message containing only one smiley. While the UI was blocked i was able to slowly count from 20 to 38. So i think it took about 18 seconds.

@ChrisLuck
Copy link
Author

After deleting the lengty conversation, the blocking-time on receive shrinked down to about a second.

@janimo
Copy link
Owner

janimo commented Dec 15, 2015

Thanks, that's good to know. No wonder I could not reproduce this, I only have short conversations.
So it is likely the naive approaches I use to show the conversation without any optimizations that are causing the lag.

@janimo janimo changed the title UI is blocking, suggest progress inidcator UI is blocking when there are many messages in the conversation Dec 15, 2015
@janimo
Copy link
Owner

janimo commented Jan 8, 2016

The qml.Changed() calls scattered in the code seem to cause a lot of overhead.

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

No branches or pull requests

2 participants