onCalendarClose is not called when selecting the same day as start&end date in selectsRange mode

[guanpu]

Describe the bug
A clear and concise description of what the bug is.
As per title, the calendar is not closed, nor is onCalendarClose being called if the startDate and endDate is the same day.

To Reproduce
try it this way:

function DateRangePicker() {
  const [startDate, setStartDate] = React.useState(Date.now());
  const [endDate, setEndDate] = React.useState(Date.now());
  const onChange = (dates) => {
    const [start, end] = dates;
    setStartDate(start);
    setEndDate(end);
  };
  const handleCalendarClose = () => console.log("Calendar closed");

  return (
    <DatePicker
      selected={startDate}
      onChange={onChange}
      startDate={startDate}
      endDate={endDate}
      shouldCloseOnSelect={true}
      onCalendarClose={handleCalendarClose}
      selectsRange
    />
  );
}

1. Open Calendar
2. select some date as startDate
3. select the same date as endDate
4. the calendar is not automatically closed, nor does the "Calendar closed" log showing up in the console.

Expected behavior
Expect calendar closed and onCalendarClose being called, just as if the startDate and endDate is not the same day.

Desktop (please complete the following information):

• OS: ubuntu 22.04
• Browser firefox
• Version 111

[balajis-qb]

This issue also exist with the onChange Event Handler when we just set the initial value for the startDate, but not the endDate like below

function DateRangePicker() {
  const [startDate, setStartDate] = React.useState(Date.now());
  const [endDate, setEndDate] = React.useState();
  const onChange = (dates) => {
    const [start, end] = dates;
    setStartDate(start);
    setEndDate(end);
  };
  const handleCalendarClose = () => console.log("Calendar closed");

  return (
    <DatePicker
      selected={startDate}
      onChange={onChange}
      startDate={startDate}
      endDate={endDate}
      shouldCloseOnSelect={true}
      onCalendarClose={handleCalendarClose}
      selectsRange
    />
  );
}

In the above case, the first onChange call won't get triggered if we select the same date for the endDate, as a few milli secs difference is there I guess.

[balajis-qb]

Hi @martijnrusschen, I think you closed this ticket accidentally.

Actually Sorry, in my PR title I wrongly linked this ticket. This is the corresponding ticket #4420. I updated the PR title now.

The PR created for this ticket is still under review only. Can you please reopen this ticket?

[guanpu]

Thank you @balajis-qb @martijnrusschen