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

Selection of multiple files #3747

Closed
predrag42on opened this issue Feb 27, 2023 · 8 comments · Fixed by #3979
Closed

Selection of multiple files #3747

predrag42on opened this issue Feb 27, 2023 · 8 comments · Fixed by #3979
Assignees
Labels
Issue-Feature A feature request or improvement.
Milestone

Comments

@predrag42on
Copy link

I can use Amaze to select files for upload (e.g. to Jira from browser), but it only offers to select one file at a time. Other file managers (e.g. default Samsung MyFiles, or XFolder) allow me to select multiple files for upload.
I'd like this feature in Amaze as well.

@VishalNehra VishalNehra added the Issue-Feature A feature request or improvement. label Feb 27, 2023
@VishalNehra VishalNehra added this to the v4.0 milestone Feb 27, 2023
@mahi072
Copy link

mahi072 commented Mar 2, 2023

Hi, I would like to work on this.
Is this issue related to flies on browser or in app, because in app we are able to select multiple files but no upload option present ?

@VishalNehra
Copy link
Member

Hi, I would like to work on this. Is this issue related to flies on browser or in app, because in app we are able to select multiple files but no upload option present ?

Great! I'm able to reproduce with following steps. Use a filr picker service such as Google drive to upload file. Select Amaze as app to select file. Then Amaze opens up, allows to upload one file by tapping on it. If you select multiple files, it should provide an option in floating action button to select multiple files to upload.

@mahi072
Copy link

mahi072 commented Mar 2, 2023

Thanks @VishalNehra, I was able to reproduce the issue.
Can you please also direct me to the exact file in this repository where the changes to be made, if possible?

@VishalNehra
Copy link
Member

VishalNehra commented Jul 12, 2023

Thanks @VishalNehra, I was able to reproduce the issue. Can you please also direct me to the exact file in this repository where the changes to be made, if possible?

sorry i missed your message here, let me know if you're still willing to contribute, I'll help you with the code change.

edit: @mahi072 ping

@seelchen
Copy link
Contributor

seelchen commented Nov 1, 2023

Hi, it has been a while without a reply from @mahi072. If no one else wants to work on this issue, I would like to work on it.
I have already started investigating a bit and found some places in the code related to this feature request.

  • MainActivityActionMode is used when multiple files are selected. The floating action button for the feature should probably be added here.
  • MainFragment#returnIntentResults is the method that returns the selected file to the other application. Currently it only takes one file, so it has to be changed to take multiple files.

I also noticed that in the Google Files app only one file can be selected if the other application only accepts one file. This should probably also be the behavior for the implemented feature.

@VishalNehra
Copy link
Member

Yea, you're on the right track. Feel free to write in case you need any clarification

@seelchen
Copy link
Contributor

seelchen commented Nov 3, 2023

Hi, after working with the code I found that there was already an attempt to implement this feature in the MainActivityActionMode.
When multiple files are selected, there is a "Select" button in the dropdown menu which should send all the selected files to the other application. The problem is that the returned Intent does not supply the URIs in a ClipData (possible since Jelly Bean) so other applications might not find the URIs in the Intent.
image

My question is if I should just fix the returned Intent of this "Select" button or if I should replace it with a floating action button.
Personally, I think that the option is too hidden in the dropdown menu. There is no indication that a new option was added so users might not find it because they only expect the usual options there.

@VishalNehra
Copy link
Member

Hi, after working with the code I found that there was already an attempt to implement this feature in the MainActivityActionMode. When multiple files are selected, there is a "Select" button in the dropdown menu which should send all the selected files to the other application. The problem is that the returned Intent does not supply the URIs in a ClipData (possible since Jelly Bean) so other applications might not find the URIs in the Intent. image

My question is if I should just fix the returned Intent of this "Select" button or if I should replace it with a floating action button. Personally, I think that the option is too hidden in the dropdown menu. There is no indication that a new option was added so users might not find it because they only expect the usual options there.

Yea, use the FAB, you can use the up arrow or upload icon in Fab for this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Feature A feature request or improvement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants