etn-ccis/blui-icons

Current needs to be converted to a filled path, not a stroke

joebochill opened this issue · 2 comments

Bug:
The Current icon does not inherit colors properly when converted to a material ui icon.

What is the expected behavior?
When setting the color for the icon, it should be applied to the icon instead of staying black.

What is the current behavior?
The icon color does not update and stays black. This is caused by the icon being created as a path with a stroke - it needs to be converted to a path with a fill.

I've built a workaround into the build-script that addresses this case, but it's rather brittle and would probably be better to enforce consistency at the design level.

Oversight on my part. Fixed in #19 (comment)