-
Notifications
You must be signed in to change notification settings - Fork 143
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
Option to max selections #313
Comments
I'll take it. Does new config maxSelections for day picker and month picker sounds good? I think that the best behaviour is when choosing max selections to disable more selections until removing a selection. |
the naming is ok. Regarding the behaviour, in my opinion there are 2 options:
I prefer the second, lets start with it? we later can add configuration to achieve the first. WDYT? |
Hi , config: IDatePickerConfig = {
// appendTo: document.body, placeholder="Select date">
</dp-date-picker>
</div>
|
@avineet1993 , the separator is set to be '|'. There is no way (currently to change it). |
@justtal I am still waiting :) |
Sorry, I'm Reacting already 🤣 😘 |
@justtal this is a step in the wrong direction :) |
In multi select mode it would be great to have maximum selections.
When selecting more than the allowed number the first selection will be unselected
The text was updated successfully, but these errors were encountered: