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

DualListBox - Added new param "Select All" to have a user select all of the availab… #1468

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

spider-bkv
Copy link
Contributor

Hi all,

Added new param "Select All" to have a user select all of the available options. Params "Min" and "Max" will be ignored.

I found it useful for others.

Thank you!

…le options. Params "Min" and "Max" will be ignored.
@ericrsmith35
Copy link
Collaborator

Are you looking to offer a Select All option to the user running the flow or are you looking to add a validation that requires the user to select every option?

@spider-bkv
Copy link
Contributor Author

Are you looking to offer a Select All option to the user running the flow or are you looking to add a validation that requires the user to select every option?

I apologize, I wasn't clear in the description.

So, the "Select All" parameter turns on a validation that requires the user to select every option.

The new parameter is located in the section "Number of Selections" on top of the "Min" and "Max" params, which get "disabled" once it's checked off. See below.

image

Thank you.

@alexed1
Copy link
Owner

alexed1 commented Oct 6, 2023

Thanks for the enhancement, Kirill!

What's your favorite example use case where the user would be asked to select everything?

@spider-bkv
Copy link
Contributor Author

Thanks for the enhancement, Kirill!

What's your favorite example use case where the user would be asked to select everything?

Thank you for the question @alexed1 !

So, the common use case description is "prioritization of a list of items" and when it's mandatory to use all of the items it's good to have the "Required to select all" parameter, instead of putting vars to Min and Max (btw, it's not possible now, also).

The most recent request I got was asking the user to prioritize a list of apartment properties (ie Floor Space, Light/Views, Closet Space, Unique Layout). After about a year in production, we've got good results from the users, it was clear to them what to do with the DualListBox moving items one-by-one from left to right in the highest-lowest order and having the ability to move them upside-down when they moved them to the right.

Thank you.

@alexed1
Copy link
Owner

alexed1 commented Nov 14, 2023

Got it... ok, I'd like to propose a few small tweaks:

  1. move your checkbox from above the Min and Max to below. That will make it clearer that Min and Max are not dependent on the setting of the checkbox.
  2. change the text to 'All items must be selected (use for sorting applications)'

@alexed1
Copy link
Owner

alexed1 commented Dec 6, 2023

Just checking in here, @spider-bkv . Do you think those tweaks would be possible?

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.

None yet

3 participants