Merge lp:~mzanetti/telegram-app/avatar-spinner into lp:telegram-app
Status: | Merged |
---|---|
Approved by: | Michał Karnicki |
Approved revision: | 162 |
Merged at revision: | 159 |
Proposed branch: | lp:~mzanetti/telegram-app/avatar-spinner |
Merge into: | lp:telegram-app |
Diff against target: |
30 lines (+6/-0) 1 file modified
telegram/app/qml/ClickableContactImage.qml (+6/-0) |
To merge this branch: | bzr merge lp:~mzanetti/telegram-app/avatar-spinner |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michał Karnicki (community) | Approve | ||
Review via email: mp+287868@code.launchpad.net |
Commit message
don't start the spinner if we don't have anything to fetch
Description of the change
This is a "fix" for the issue that victor found in QA today.
The issue was that when clicking the avatar of some contact details page (or the settings page), the app tries to load the big image from the network and displays a spinner for that. Now the problem was, that this was triggered even when there is no profile picture, in which case the spinner just keeps on spinning forever. In the account settings page, this even suggests that it would try to open content hub for importing a picture and keeps the user waiting forever.
We could probably improve this and actually open the content hub, however, this component is used also when you view another contact's details in which case we can't set the image so it's not 100% straight forward. This patch at least doesn't start the spinner if we don't have anything to fetch from the network and doesn't keep the user waiting.
Importing a new avatar is, after all, done from the toolbar icon for that.
Thank you :D