-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
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
Fixed DatePicker year/month navigation buttons #1081
Conversation
Addresses #1075 |
}, style); | ||
}, | ||
style, | ||
{fill: this.state.hovered ? onColor : offColor}); // Make sure our fill color overrides fill provided in props.style |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a small code formatting suggestion:
let mergedStyles = this.mergeAndPrefix({
display: 'inline-block',
height: 24,
width: 24,
userSelect: 'none',
transition: Transitions.easeOut(),
}, style, {
// Make sure our fill color overrides fill provided in props.style
fill: this.state.hovered ? onColor : offColor
});
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You got it, I'll make the change.
@hai-cea Updated the formatting. |
@jkruder Please rebase. |
…s.style. Also fixed zindex issue with the month/year.
@hai-cea Rebased. |
Fixed DatePicker year/month navigation buttons
Thanks @jkruder |
In svg-icons there was a command to delete the fill property from this.props.style. This is destructive behavior on something that should be treated as immutable. This issue presented itself as soon as I hovered over one of the year/month buttons in the DatePicker that was disabled.
Also fixed a z-indexing issue with the month year being displayed between the month/year buttons.