Fix bug in translating 'primaryFill' to 'color' for font icons

microsoft / fluentui-system-icons

Fluent System Icons are a collection of familiar, friendly and modern icons from Microsoft.

https://aka.ms/fluentui-system-icons

MIT License

5.91k stars 516 forks source link

Fix bug in translating 'primaryFill' to 'color' for font icons #472

Closed MLoughry closed 2 years ago

MLoughry commented 2 years ago

useIconState translates primaryFill to fill in the returned object. The typing on the returned object of primaryFill: string | undefined is technically not wrong, but was misleading.

This change fixes the intended logic to read primaryFill directly from the props.