[pickers] onMonthChange doesn't call after click to days outside the current month

[Semigradsky]

Duplicates

• I have searched the existing issues

Latest version

• I have tested the latest version

Steps to reproduce 🕹

Link to live example: https://codesandbox.io/s/zen-hodgkin-8ijm8m?file=/demo.tsx

Steps:

1. Add showDaysOutsideCurrentMonth
2. Click to day outside current month
3. Only onChange callback was called. onMonthChange wasn't called.

Current behavior 😯

onMonthChange wasn't called.

Expected behavior 🤔

onMonthChange called.

Context 🔦

By documentation:

onMonthChange - Callback firing on month change

Your environment 🌎

No response

Order ID 💳 (optional)

No response

[flaviendelangle]

Hi,

Indeed, onMonthChange is only fired when you change the value from the outside of the picker or if you change it from the month view.

That's not very coherent, I'll look on how to improve it 👍

[dhulme]

I have a similar issue that the onMonthChange doesn't fire when using the keyboard to navigate between months. Not sure if your PR would coever than too @flaviendelangle?

[flaviendelangle]

This PR was not merged yet because we are unclear about what the callback should designate (should it be fired when the month of the value changes or when the month visible on screen changes).

If it's the 2nd one, then sure the PR will cover your use case.

If it's the 1st one, we will probably add a new callback for the visible month on screen.

[dhulme]

Ok thank you. Perhaps keeping the current callback for the value, and adding a new onVisibleMonthChange or similar could be a solution?

[flaviendelangle]

That's one of the approach we can take indeed