> LGTM. Is this code covered by tests? Should it be?
IconRenderer is test free :/ Btw this is a visual change and we don't need test for visual changes. I'll ask Tim btw.
« Back to merge proposal
> LGTM. Is this code covered by tests? Should it be?
IconRenderer is test free :/
Btw this is a visual change and we don't need test for visual changes. I'll ask Tim btw.