search

arttuka / reagent-material-ui

Reagent wrapper for MUI (formerly Material UI) v5
Eclipse Public License 2.0
203 stars 18 forks source link

Icons should be in a single namespace #33

Closed conjurernix closed 2 years ago

conjurernix commented 2 years ago

For the love of god, please add icons in a single namespace.

arttuka commented 2 years ago

Due to the way Clojurescript is compiled, this would be a very bad idea. If all the icons were in the same namespace, the code for all the icons would also be included in any build that uses even one icon. I tested this and it would add 4.7 megabytes of JS code to all builds (after optimization).