Android: Auto upload: Screenshots folder should have priority sorting

[jancborchardt]

Currently there’s two folders important for me to auto-upload: Camera and Screenshots.

Camera is correctly sorted up top. Screenshots however is all the way on the bottom between folders of album covers and such on the SD card. It should be given priority sorting and put to the top, even when not synced.

What do you think @tobiasKaminsky @AndyScherzinger?

[mario]

Makes no sense to me? Why would Screenshots get preferential treatment? Afaik Camera is at the top because we sort by (IMHO correct) by alphabet.

[AndyScherzinger]

I'd also vote against it since for me Screenshots is the least important folder to me.

Currently there’s two folders important for me to auto-upload: Camera and Screenshots.

This is imho a personal preference, I for example care about Camera, Instagram and WhatsApp

Afaik Camera is at the top because we sort by (IMHO correct) by alphabet.

• Kind of... We sort alphanumerically (by alphabet) but give "Camera" a priority since we "decided" that for auto upload this would be the most important folder ever
• At the very top are always the active one, so if someone would think the Screenshots are important than activate auto upload and it will be on top. If it is not important enough to be auto uploaded it is imho not important enough to be sorted at the top when inactive. "Camera" imho being the only special folder we should have.

2 two cents :)

@tobiasKaminsky ?

[jancborchardt]

I wasnt talking about _top_ priority. The current state though is that I have around 50-100 other folders in that list I need to scroll past - and they are all irrelevant to autoupload since they are album covers from my music.

So this, and WhatsApp, other specific widely used ones should definitely be specifically sorted before any random folders with images.

[mario]

@jancborchardt this explanation makes more sense, but it's rather tricky as we can also have "Snapchat" folder without it actually belonging to Snapchat. Edge case, I know, but still...

[AndyScherzinger]

@jancborchardt kind of different approach I wanted to implement / suggest at some point anyways: allow the user to hide folders in the auto upload view.

Would that be something that should be implemented and would this work around this issue? I know it is not as nice as an intelligent sorting while I don't know how to actually implement that since we would then need a list of typical app folders that need priority while most likely missing some for a certain user base I guess :/

[mario]

Oh my. This further complicates it...

On Tue, Sep 5, 2017 at 9:07 PM, Andy Scherzinger [e-mail]
wrote:

@jancborchardt https://github.com/jancborchardt kind of different
approach I wanted to implement / suggest at some point anyways: allow the
user to hide folders in the auto upload view.

Would that be something that should be implemented and would this work
around this issue? I know it is not as nice as an intelligent sorting while
I don't know how to actually implement that since we would then need a list
of typical app folders that need priority while most likely missing some
for a certain user base I guess :/

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nextcloud/android/issues/1493#issuecomment-327273071,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAWsljRlYvhUdWIntinEvUA_l6o7XcEks5sfZv4gaJpZM4PMzWG
.

[AndyScherzinger]

xD

Just a thought ;) No worries :D

[comradekingu]

@jancborchardt How about being able to scroll upwards to get to the latest ones, (bottom of list) if sorted by date?
It would at least give quick access to two places.

[jancborchardt]

allow the user to hide folders in the auto upload view.
Would that be something that should be implemented and would this work around this issue?

No, as it doesn’t change the default. ;)

I had no idea this would be such a big discussion. In my experience there are _two_ picture folders in a default Android install (without any additional apps) which can have stuff which is only on your phone and you would need to back up. One is Camera, the other Screenshots. We do list Camera first correctly already, so putting Screenshots below seems sensible.

[mario]

There are no folders by default. They might EXIST, but not registered as media providers. Not super excited about this, but not against it much either. If @AndyScherzinger is for it, we can do it.

[tobiasKaminsky]

I understand the jan's intention, but this is a config screen which is not accessed that often.
So I am fine with both

• make camera and screenshot top priority
• keep it the way it is.

If we change it, this should be relative simple...
Extend it to a list and voila ;-)
https://github.com/nextcloud/android/blob/329bccd51439705520e153cd53aec21ae4aa8e58/src/main/java/com/owncloud/android/ui/activity/SyncedFoldersActivity.java#L89-L89

[tilosp]

What is the state of this?
@AndyScherzinger Are you fine with prioritizing the screenshots folder?

[AndyScherzinger]

@tilosp still pending I'd say. I have the same opinion as Tobias https://github.com/nextcloud/android/issues/1493#issuecomment-336435326 -> I'm fine either way

[AndyScherzinger]

Thanks for the contribution @tilosp 🥇 🎉

[jancborchardt]

Thanks a lot @tilosp! Great contribution! :)