Open
Description
See previous open issue here: #224
Needed to open another ticket to track towards making this change for v0.16.0
.
React has changes it type definitions which means React.FC
will no longer be available to use.
- Update each of the icons to use the appropriate props
- [ ] Consider usingSvgIcon
from@mui/material
as it has other properties we can benefit from usingWiden theIconProps
to acceptsx
props from MUI in order to utilize the theme object inline
- 🎨 Wireframes and designs for Layer5 site in Figma (open invite)
Activity
[-]Refactor icons to not use`React.FC`[/-][+]Refactor icons to not use `React.FC`[/+]senthil-athiban commentedon Apr 8, 2024
@nebula-aac Should I refactor all the SVG icons to use the
SvgIcon
component frommui
?nebula-aac commentedon Apr 9, 2024
@senthil-k8s Hello thank you for your question
Initially I considered doing this, because when I was testing it in the open PR I had for testing new changes to move to Next.js 14, there are underlying issues with using the MUI svg api to replace ours, which can't be done all at once, but rather needs to be one by one. So this issue needs to be changed to reflect the possible blockers.