update menu sidebar to use official Radius icons #50
Conversation
Signed-off-by: Will Tsai <[email protected]>
|
I'm not satisfied with how it looks currently, so I've converted this to draft for now while I figure out how to resize the icons: 
|
|
This is awesome 😍 |
| viewBox={ | ||
| props.shape === 'square' | ||
| ? `0 0 ${originalHeight} ${originalHeight}` | ||
| : `0 0 ${originalWidth} ${originalHeight}` |
There was a problem hiding this comment.
Do you need all of the complexity here with .shape? This was needed for the "Radius" logo because the text is part of the graphic.
There was a problem hiding this comment.
I kept this for in case we evolve the icons into non-square shapes in the future, but I get your point that it seems unnecessary, so I'll remove the .shape property since we don't really need it at this point.
Signed-off-by: Will Tsai <[email protected]>
…ard-radius into use-radius-icons
After removing the 
|
Signed-off-by: Will Tsai <[email protected]>
|
Thanks @willtsai ! |
fixes: #43