Skip to content
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

Pass nextMonth to prev and next month click props #1207

Conversation

mmahalwy
Copy link

No description provided.

Copy link
Collaborator

@majapw majapw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM once the prettier changes are removed!

newMonth.subtract(1, 'month');
if (onPrevMonthClick) onPrevMonthClick(newMonth);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is exactly what I was going to suggest. :)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

haha nice!

@mmahalwy
Copy link
Author

ok, removed the prettier changes!

@coveralls
Copy link

coveralls commented Jun 12, 2018

Coverage Status

Coverage remained the same at 85.179% when pulling 33b6827 on mmahalwy:me--pass-moment-for-prev-next-click-day-picker into ef42df2 on airbnb:master.

@ljharb ljharb added the semver-minor: new stuff Any feature or API addition. label Jun 12, 2018
@mmahalwy
Copy link
Author

good to merge?

@majapw majapw merged commit a944148 into react-dates:master Jun 13, 2018
@mmahalwy mmahalwy deleted the me--pass-moment-for-prev-next-click-day-picker branch June 13, 2018 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-minor: new stuff Any feature or API addition.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants