Address DayPickerNavigation rerender

[majapw]

Fix for the DayPickerNavigation rerender discussed in #1297

Pretty simple fix involving moving around some arguments.

to: @ljharb @lencioni @AntiFish03

[ljharb]

it's a pretty strong convention to have e be last; why is this change needed?

[majapw]

The motivation for this was because we were creating a new function in every render and causing unnecessary updates as a result. Flipping the order of arguments was a pretty simple solution, and was motivated by the fact that both arguments are occasionally omitted.

We could alternatively cache the function with the event as the second arg as well if you think it makes more sense @ljharb. It may make sense to have onNextMonthClick take the event as an argument and call a function like transitionToNextMonth that can also be called directly. 🤔 Actually, that may be less confusing in the long run

[coveralls]

Coverage increased (+0.06%) to 84.978% when pulling bffe704 on maja-address-daypickernav-rerender into f46195f on master.

[majapw]

@ljharb I've split out the two cases that call the month transition into two functions. I think this approach makes more sense! Can you take another look?