search

Esri / calcite-colors

Esri's official color documentation repository that is leveraged by Calcite and all other Esri initiatives.
https://esri.github.io/calcite-colors/
Other
27 stars 7 forks source link

Invalid css color #73004 #75

Open ek-work opened 2 years ago

ek-work commented 2 years ago

Shouldn't the color be #73004c instead of #73004? Reference: https://github.com/Esri/calcite-colors/blob/1eb6e5cb041eb55d25b5f7c13c375b2788ef256f/src/colors.ts#L114

ek-work commented 2 years ago

@driskull Hi Matt - not sure if this is the right place to submit this issue. But it seems this color should be #73004c rather than #73004. Can you confirm?

driskull commented 2 years ago

@asangma @macandcheese @paulcpederson could you comment on this? I have no idea on the colors.

macandcheese commented 2 years ago

Yeah, seems like that's missing a chracter. Not sure the origin of these particular values but the update above makes sense to fix to me.

paulcpederson commented 2 years ago

Yeah looks like it was added originally this way so no idea what that value was supposed to be...

@kyle-03674 do you know if we have a color value for the charts colors where the hex begins with #73004 ? Or where do we pull the charts colors from, the vibrant set?

kyle-03674 commented 2 years ago

I was just looking at that. I didn't have that color value in any of my work for the core color palettes. I think this was a chart-specific color.