Case of string values.

[0rion7st]

In case when value is {"red":"255","green":"255","blue":"255","alpha":"0.5"}
Properties are not Numbers function: _setValueAsHexFromValue works not correct.

Recommended fix:
paper-color-input.html:206
var h = parseInt(value[c]).toString(16);

[David-Mulder]

Honestly, why are you setting it to strings in the first place? the parseInt shouldn't happen inside the component, but outside it :/ .

[0rion7st]

As I said, it's a recommended, I'm using this element in 2 different scenarios, in both this check is "good to have". Anyway even if you don't want to parse. Check with Number.isInteger in order not to produce errors in javascript.

Probably I'll not to fork your repository in order to add additional configuration and bug fixes. I'll try to make some pull request for your judgement.
Thanks for awesome element anyway 👍

[David-Mulder]

@0rion7st Well, there are a million other ways developers could give incorrect input as well, so I am not going to check every attribute whether it's valid as that's a waste of runtime.

BUT there is a big bug caused by the lack of a valueAsNumber property on paper-input[type=number] for which I send a PR already weeks ago: PolymerElements/iron-input#68 If that's the problem you're encountering then yeah, I need to somehow fix that in the meantime :/ I was hoping the PR would get pulled quickly, but it's taking ages.