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 Uploader #1038

Closed
tobiasKaminsky opened this issue Jun 28, 2015 · 10 comments
Closed

Improve Uploader #1038

tobiasKaminsky opened this issue Jun 28, 2015 · 10 comments
Labels

Comments

@tobiasKaminsky
Copy link
Contributor

From #903 (comment):
What about some kind of "Upload new" window in the app, which isn't part of instant upload, but basically:

  1. Shows thumbnails of all new photos and videos since the last upload
  2. Allows users to preview each
  3. Allows easy multi-selection and
  4. Uploads all selected with a single click.

@Bugsbane that sounds like a pretty clever thing. Perhaps you can even just take the current upload dialog and let it hide things (behind a 'previously uploaded' button) which are already on the ownCloud server...

Even better. Maybe a "Hide previously uploaded" checkbox or something.

@Bugsbane
Copy link

Yes, I think it's an excellent idea to just add functionality to the current upload window, rather than creating a second one.

From a UX perspective, I would suggest that in the vast majority of use cases, users are only likely to be wanting to upload things they haven't already uploaded before. In those cases it would be one less interaction step for users if previously uploaded files are hidden by default, as long as it is made very clear what is happening and how to see the full list if needed.

I'd suggest that what is currently the file selector window have a toggle switch at the very top labelled "Show all files". By default it would be off, with two statuses: "Only show files which have not been uploaded" and "Show all files, including previously uploaded." Underneath would be the file list, like this:

new-uploads-1

Another alternative that I don't like as much, but would take less screen space up top, would be to hide the toggle in an options (3 dot) menu, and then for people who didn't think to look in there, down the bottom of the list of files, we could add a button that said "Show all files / Including files already uploaded." Presumably, anyone who couldn't see their file would scroll to the bottom of the list and see this button. I don't think this option is as clear as my first suggestion, but it does save on screen real estate, leaving more room for the file list, so I'm adding it here anyway:

new-uploads-2

As I said before, my preference would be the first suggestion. It's more consistent with Android UI and clearer to users who don't know the option exists.

@jab416171
Copy link
Member

I prefer the first option. We should try to stay as close to material design as possible.

I'm not sure if any of you have used the Photos app in Lollipop, but I really like its interface. There's an option in the drawer to see "Device folders", and from there, you can choose to back up the entire folder with a toggle per folder. This would be great for the folder that holds all of your pictures.

Additionally, under "Backup & sync", there's toggles to back up while roaming, while charging, and while on Wi-Fi, and a "Back up all photos & videos now (if connection settings allow). I realize this is just for backing up photos & videos, but we could probably adapt most of the UX (Simply taking out "Photos" or "Videos" would probably work just fine) to work with any folder.

@jab416171
Copy link
Member

Here's some screenshots from the Photos app:

device-2015-06-28-113805

device-2015-06-28-113855

@Bugsbane
Copy link

@jab416171 - There's another issue filed somewhere on here that's specifically about backing up and syncing multiple folders to ownCloud. Your comments are probably more relevant on that one if you can find it, than on an issue about uploading images not yet uploaded via the file selection window.

@jab416171
Copy link
Member

alright, thanks @Bugsbane. I didn't realize that.

@masensio
Copy link

This issue contains:
#1015
#899

@tobiasKaminsky
Copy link
Contributor Author

@Bugsbane You mean #744 ?

@Bugsbane
Copy link

I think so, yes.

@roy128
Copy link

roy128 commented Aug 3, 2015

👍

@jesmrec
Copy link
Collaborator

jesmrec commented Mar 31, 2022

Native picker is used currently.

@jesmrec jesmrec closed this as completed Mar 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants
@jab416171 @Bugsbane @masensio @tobiasKaminsky @roy128 @jesmrec and others