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

Nearby -> System picker: Only allow 1 picture to be selected #5677

Closed
nicolas-raoul opened this issue Mar 30, 2024 · 6 comments · Fixed by #5680
Closed

Nearby -> System picker: Only allow 1 picture to be selected #5677

nicolas-raoul opened this issue Mar 30, 2024 · 6 comments · Fixed by #5680

Comments

@nicolas-raoul
Copy link
Member

I think we can specify whether we want 1 file or several, when we call the system picker intent.

Similar to #5674

@rohit9625
Copy link
Contributor

Hello, Mr. Nicolas :)
I made a minor change to prevent multiple selections of pictures in System Picker. This is what it behaves like now:

Nearby_Custom_Picker.mp4

Is this the required behavior, please explain if I misunderstood.
But for Custom Picker It would require a complete implementation to achieve this and I will start working towards it soon.

@nicolas-raoul
Copy link
Member Author

Looks perfect! Do you want to submit a PR?
You are right, it will require some custom code for the custom picker. And passing an argument when calling it.

@rohit9625
Copy link
Contributor

Yes for now, I am submitting PR for this one.

@rohit9625
Copy link
Contributor

Here when passing binding.nearbyFilter.chipView.choiceChipWlm.isChecked() to the function initiateGalleryOick:
image

I wonder why the check WLM is not shown in the Nearby filters? I need to pass false for the required behavior.

@nicolas-raoul
Copy link
Member Author

WLM only happens at certain times of year I believe, there might be a check for that. Don't worry too much about WLM for now.

@rohit9625
Copy link
Contributor

Okay then, I got it what to do now :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants