You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I have seen good implementation on the #344 which is awesome to work with, but the onActiveStartDateChange does not give any clue which option has trigger active date to be changed on the calendar.
I would suggest to give additional option in function to give flexibility to set activeStartDate or prevent start date to be changed by clicking on the prev month dates. This is really annoying to me when user click on prev month dates where I dont want it to jump to the prev month but to stay in it, as specially in date range where dates has to be selected in from 30 - 2 (too many clicks)
onActiveStartDateChange={({ activeStartDate, value, view, handlerType }) => {
// handlerType can be
// prevLabel - button on the navigation pane
// nextLabel - button on the navigation pane
// prev2Label - button on the navigation pane
// next2Label - button on the navigation pane
// prevMonth - prev month dates of Dec 27 - 30
// nextMonth - next month dates of Feb 1 - 6
if (handlerType === nextMonth) {
return;
}
return setActiveStartDate(activeStartDate)
}}
Unless you think to that have this build in to just boolean prop so that it easy on of when it needs
The text was updated successfully, but these errors were encountered:
I have seen good implementation on the #344 which is awesome to work with, but the
onActiveStartDateChange
does not give any clue which option has trigger active date to be changed on the calendar.I would suggest to give additional option in function to give flexibility to set activeStartDate or prevent start date to be changed by clicking on the prev month dates. This is really annoying to me when user click on prev month dates where I dont want it to jump to the prev month but to stay in it, as specially in date range where dates has to be selected in from 30 - 2 (too many clicks)
Unless you think to that have this build in to just boolean prop so that it easy on of when it needs
The text was updated successfully, but these errors were encountered: