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

Improve cloud project download/synchronization feedback #5050

Merged
merged 8 commits into from
Feb 24, 2024
Merged

Conversation

nirvn
Copy link
Member

@nirvn nirvn commented Feb 23, 2024

This PR improves the cloud project download/synchronization UX by providing better feedback to users, including:

  • Having a functional progress bar reflecting download % in the cloud projects list (until now, it was always indefinite)
  • Adding a progress bar in the cloud project popup when hitting synchronize
  • Making a better job at differentiating packaging job waiting vs. initial communication to server vs. download (on the projects list, due to a broken logic, it could be stuck never reaching the download % feedback)
  • Adding total download size detail (e.g., Downloading, fetched %10 of 200MB).

In action:

Screencast.from.2024-02-23.16-24-11.webm
Screencast.from.2024-02-23.16-53-46.webm

With these in, when access to the cloud server is slower are likely to find the wait more tolerable, and at least will not have the impression that QField is "frozen".

@qfield-fairy
Copy link
Collaborator

qfield-fairy commented Feb 23, 2024

default:
status = qsTr('QFieldCloud is preparing the latest data just for you. This might take some time, please hold tight…')
break;
if (PackagingStatus === QFieldCloudProjectsModel.PackagingBusyStatus) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ideally we can avoid repeating this as it is already in QFieldCloudPopup.qml bit I cannot suggest anything that is easy and worth it.

@nirvn nirvn enabled auto-merge February 24, 2024 11:06
@nirvn nirvn disabled auto-merge February 24, 2024 13:57
@nirvn nirvn merged commit 2de4b70 into master Feb 24, 2024
22 of 23 checks passed
@nirvn nirvn deleted the cloud_feedback branch February 24, 2024 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants