RaisedButton,Use labelColor property,The button will not trigger the hover state

[likun7981]

Problem description

RaisedButton,Use labelColor property,The button will not trigger the hover state

Versions

• Material-UI: 0.15.2
• React: 15.2.1
• Browser: Chrome 51.0.2704.103 (64-bit) Mac

[gyfis]

I've just encountered this bug as well, though I need to say that for me, it didn't work when I specified the color by name (e.g. labelColor={'white'}), but when I replaced the color with actual hex code (e.g. label={'#ffffff'}) it started to work again - I think it's because the hover is created by adding alpha to the color and the framework not being able to properly parse RGB from the string color representation - this means that it can be fixed on user end but it might affect more components.

[mbrookes]

@gyfis That's correct, we don't attempt to map string colours in https://github.com/callemall/material-ui/blob/master/src/utils/colorManipulator.js.

Given the move to jss styling in the next branch I think it's unlikely we'll fix this, so am closing the issue.

@gyfis Thanks for following up.

[gyfis]

@mbrookes Thanks for the info! Yeah, it makes sense not to parse colors input in this way. Do you think it would be a good idea to add a general warning about 'adding a non-parseable color to component - this might cause component effects not to work', possibly in the next branch?

[mbrookes]

We don't have xxxColor props on next, so it will be less of a problem. Throwing a warning in the decomposeColor function in colorManipulator on master mightn't be a bad idea.

Feel free to submit a PR - should only need a couple of lines. Here's an example warning: https://github.com/callemall/material-ui/blob/master/src/DatePicker/dateUtils.js#L11

[gyfis]

I submitted the PR: #5939