apps-android-commons icon indicating copy to clipboard operation
apps-android-commons copied to clipboard

Long-press on a folder to reveal a menu offering folder operations

Open nicolas-raoul opened this issue 1 year ago • 9 comments
trafficstars

The app has 3 ways to upload new pictures, one of them is the "custom picker". The custom picker has an activity that shows a list of folders:

Screenshot_20241009-122706.png

The present issue is about that custom picker's folders list activity.

The idea is to implement long-press on a folder to reveal a menu offering folder operations:

Phase 1:

  • Open the folder in another app (just launch an intent, Android should hopefully suggest available file explorers and gallery apps).

Phase 2:

  • Mark all pictures currently in this folder as "Not for upload".

Phase 3:

  • Always ignore this folder (which makes it greyed out, with 🚫 overlay icon, shown at the end next time the picker is opened)
  • Un-ignore if the folder if it is the "Ignored" state.

nicolas-raoul avatar Sep 03 '24 09:09 nicolas-raoul

These use cases are very good but I don't think the user wants to open the folder in another app when the user can open it just by clicking. However, other features are great :)

rohit9625 avatar Sep 04 '24 06:09 rohit9625

Opening the folder in a file manager may reveal other files, such as videos (which can be uploaded via other means), and possibly memo text files which might add useful context to the pictures.

nicolas-raoul avatar Sep 04 '24 06:09 nicolas-raoul

Okay, now I got it :+1:

rohit9625 avatar Sep 04 '24 06:09 rohit9625

Hi, Can I please be assigned this issue

LachlanMajor avatar Oct 09 '24 07:10 LachlanMajor

Well currently the custom selector is in xml and I am working on migrating it to the jetpack compose. So, any changes will be redundant as for now. If you want to work on this feature then you can do so after the migration. I will open a draft PR for that shortly.

Kindly look for issues unrelated to custom selector.

What do you think @nicolas-raoul ?

rohit9625 avatar Oct 09 '24 08:10 rohit9625

@rohit9625 Sounds good! Would you mind pinging LachlanMajor here after the merge? :-)

nicolas-raoul avatar Oct 09 '24 08:10 nicolas-raoul

Sure, I will ping @LachlanMajor after migration. 🙂

rohit9625 avatar Oct 09 '24 09:10 rohit9625

@LachlanMajor Feel free to pick another issue meanwhile. Thanks! 🙂

nicolas-raoul avatar Oct 09 '24 10:10 nicolas-raoul

Yeah thanks Ill look for another one to do instead.

LachlanMajor avatar Oct 09 '24 11:10 LachlanMajor