Introduce Lucide based Icon component and updates#209
Conversation
01666b0 to
ab960fa
Compare
492ae7a to
5c56b01
Compare
88f577a to
0190d4c
Compare
|
I was expecting this to just replace the icons line by line. Can you explain what the additional complexity is trying to achieve? (in particular, all the files in DataDisplay/Icons) |
As far as I understand, it's better to run Lucide Icons through Key icons will be in the theme, but if an app wants to use other icons they can import it from Lucide, and then use |
0190d4c to
dc92db2
Compare
33737d7 to
b3dfd34
Compare
- Added Lucide Icons as basis for most places - Updated stories and components with relevancy to icons - Removed dependency for React Icons. - Removed reference to unused icons *We can also Remove `@mui/icons-material` but not sure of others may use it
7d18fa5 to
0d22e85
Compare
react-icons@mui/icons-materialreact-iconspackageNote: We can also safely remove
@mui/icons-materialpackage but not sure of others may use it