android icon indicating copy to clipboard operation
android copied to clipboard

File removal dialog offers too many choices

Open timmersr opened this issue 7 years ago • 9 comments

I am always a bit anxious when removing a downloaded file from my phone, because I have to select 'only local' in between two other options, in order to leave it on the server.

I am only looking to remove it from my phone, and never from the server.

Could you create this as selectable setting for default removals?

timmersr avatar Dec 11 '18 09:12 timmersr

GitMate.io thinks possibly related issues are https://github.com/nextcloud/android/issues/1588 (Add customizable delay on removal of original file), https://github.com/nextcloud/android/issues/411 (Removal failed), https://github.com/nextcloud/android/issues/2522 (Set/unset favorite should refresh file sorting), https://github.com/nextcloud/android/issues/1877 (Files not selectable via the checkbox), and https://github.com/nextcloud/android/issues/2919 (Sort Deleted files by "Deletion date" by default).

nextcloud-android-bot avatar Dec 11 '18 09:12 nextcloud-android-bot

@tobiasKaminsky @timmersr I strongly vote no.

We do have the trashbin so the files (if deleted accidentially from the server) can be restored and I wouldn't want to start implementing UI flags to have a customizable UI on a per-user level available since this doesn't add real value to the app but a lot of complexity that needs to be maintained in the future.

AndyScherzinger avatar Dec 11 '18 09:12 AndyScherzinger

2018-12-11-110514

I fully agree with Andy. What we can do is to highlight "local only" as this is probably the most used option, and the other one in grey.

tobiasKaminsky avatar Dec 11 '18 10:12 tobiasKaminsky

@nextcloud/designers

tobiasKaminsky avatar Dec 11 '18 10:12 tobiasKaminsky

We had also the idea to offer an undo on deletion as snackbar. This will give the user a bit of confidence that nothing really can go wrong…

tobiasKaminsky avatar Dec 11 '18 10:12 tobiasKaminsky

Right, this is a good point to raise, I also really would like to simplify this dialog. Just to start, no we will not add a setting for this.

What we can do is to highlight "local only" as this is probably the most used option, and the other one in grey.

We don’t have a way to really make an informed decision here through metrics, right? Because when I delete, I basically always mean delete from the server. Don’t care about phone space even though it’s not huge storage.

This is also how Google Drive and Dropbox do it. They don’t have a specific "Local only" deletion. Files simply get removed, and you get a Snackbar confirmation "Filename deleted [ Undo ]". It’s seamless and not scary, and since we have deleted files, you can restore them at any point too.

So for people who like to keep an eye on their phone storage, what we could do is have an entry somewhere to clear the downloaded files? Probably in the settings. Then again, Android offers these ways of cleaning data of apps.

jancborchardt avatar Dec 21 '18 02:12 jancborchardt

This is also how Google Drive and Dropbox do it.

This is because they have all files only available in cache. They do not expose the downloaded files the way we are doing it. So we cannot directly compare it.

So for people who like to keep an eye on their phone storage, what we could do is have an entry somewhere to clear the downloaded files?

What if you want to delete only a part of your downloaded files? E.g. remove all downloaded music, but keep the vacation pictures?

Android offers these ways of cleaning data of apps.

We already have this, but it deletes all files from all accounts…

tobiasKaminsky avatar Dec 21 '18 12:12 tobiasKaminsky

This is because they have all files only available in cache. They do not expose the downloaded files the way we are doing it. So we cannot directly compare it.

Ok, but that’s just how it’s implemented. From the user perspective this doesn’t matter that much.

What if you want to delete only a part of your downloaded files? E.g. remove all downloaded music, but keep the vacation pictures?

Good point. So then we should have a thing in the settings called

Clear downloaded files Choose which to keep and which to remove

And it will show basically a folder-picker dialog with checkboxes where by default everything is selected, and two buttons at the bottom "Remove local files" and "Cancel". And there you can unselect what you like.

What do you think?

jancborchardt avatar Dec 21 '18 13:12 jancborchardt

Good point. So then we should have a thing in the settings called

Clear downloaded files Choose which to keep and which to remove

What do you think?

It might make sense to put this sort of thing under the existing On Device tab since that's where all the downloaded files are already centralized.

I am always a bit anxious when removing a downloaded file from my phone, because I have to select 'only local' in between two other options, in order to leave it on the server.

At least today this is somewhat alleviated because Delete is now a distinct button visually distinguishing it from the adjacent Local only. Also, as already noted, there's the trash bin so accidental deletions are not immediately permanent (though I see how anxiousness might continue since you may not discover the evidence of what button you pushed until weeks/months later).

Personally I find the shifting of the Keep option on the Delete dialog for downloaded files jarring:

Screenshot_20231219-153921.png

Screenshot_20231219-153701.png

I'd prefer the Keep action stayed in the same place for either file type so the UI is always consistent and muscle memory friendly, and Local only gets added at the far left for Downloaded files instead of where Keep gets shifted to for those.

It also would have the benefit of Local only no longer being stuck being two other options - the original concern brought up in this Issue.

joshtrichards avatar Dec 19 '23 20:12 joshtrichards