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

Allow smoother Single image selection #133

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

omaraltayyan
Copy link

when a single image selection mode is selected (by setting imageLimit to 1) the controller doesn't select the image until the user manually deselects the previous one, this modification automatically removes the last picked image to allow for the new image to be selected in single image selection mode.

when a single image selection mode is selected (by setting imageLimit to 1) the controller doesn't select the image until the user manually deselects the previous one, this modification automatically removes the last picked image to allow for the new image to be selected in single image selection mode.
@3lvis
Copy link
Collaborator

3lvis commented Oct 4, 2020

Hi @omaraltayyan, sorry for the lack of follow up, Gallery was entered into maintenance mode and didn't have someone following up issues. Since it's been a while since you submitted this I'm wondering if it's still relevant.

Thanks for your time and have a nice week.

@omaraltayyan
Copy link
Author

Hi @omaraltayyan, sorry for the lack of follow up, Gallery was entered into maintenance mode and didn't have someone following up issues. Since it's been a while since you submitted this I'm wondering if it's still relevant.

Thanks for your time and have a nice week.

Hi, thanks for the reply, i actually changed my entire field of work when that PR was posted, if u find the code to make sense or need modification feel free to work on it, i wish u all the best

@3lvis
Copy link
Collaborator

3lvis commented Oct 4, 2020

I understand, crazy how much can happen in 2 years. Take care.

@3lvis 3lvis closed this Oct 4, 2020
@omaraltayyan
Copy link
Author

@3lvis i remembered something about this PR, we actually tested this and deployed it, it makes the behavior much better, so now if the limit of the selection is 1, you get a "Radio Button" like behavior, deselecting the previous image when a new one is selected, previously it required the user to deselect the image manually before selecting a new one, this fixes its behavior very well.

@3lvis 3lvis reopened this Oct 6, 2020
@3lvis
Copy link
Collaborator

3lvis commented Oct 6, 2020

@omaraltayyan I'll take a look, thanks!

@HB-Appinneers
Copy link

Hi is this implemented as I too was looking someway if only one image selection is allowed then it should be like when a new image is selected the previous one should get deselected.

@omaraltayyan
Copy link
Author

Hi is this implemented as I too was looking someway if only one image selection is allowed then it should be like when a new image is selected the previous one should get deselected.

this was working when I implemented it, i don't know if they changed this code now or not, but i remember we tested it thoroughly and it was working pretty well

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

Successfully merging this pull request may close these issues.

3 participants