Piwigo / Piwigo-Android

Piwigo Native Android App
GNU General Public License v3.0
140 stars 43 forks source link

Improve "Upload" functionnality #118

Closed Valou447 closed 5 years ago

Valou447 commented 5 years ago

When you upload some photos, everything runs in background, and the user is not really informed of what is going on after picking his photos.

Some good idea would be to add a ProgressCircle in a dialog when the user is in the app and a notification with a progress bar to keep him informed about how much photos have been uploaded and how many are left.

Also, we need to refresh the image list for the current album, when the upload is done.

ramack commented 5 years ago

Definitly an important topic! (See #40) A separate progress circle.is.maybe not.the best option, but as you anyhow plan.to add a FAB for uploading (#40 #104) it might be an option to make the spinning circle around that one in the app. For me it would also be sufficient to have only the notification with a progress bar as used for downloads. Maybe https://medium.com/@britt.barak/notifications-part-2-progress-indicator-2aa4cdea24c6 can help...

Valou447 commented 5 years ago

Something we are missing in the app atm is telling the user that something is happening. Everything runs in background and the "Uploading.." toast does not even show up. A toast to inform the user that the upload is done is shown but the photos does not show up neither, so we are facing an issue right there.

A progress bar linked to the FAB is a pretty good idea tbh, because just sending a notification when the user is already in the app is not, to me, enough.

ramack commented 5 years ago

I fully agree and think we should even get rid of those toasts completely and replace longer status messgaes (like upload progress) in a notification, while for events snackbars are more appropriate.

Valou447 commented 5 years ago

I just figured out that picking more than one photo in the chooser works, but then only 1 is uploaded.

Valou447 commented 5 years ago

Done in #127.