-
Notifications
You must be signed in to change notification settings - Fork 2
Description
Is your feature request related to a problem? Please describe.
I looked into upgrading the existing MUI v4 styles to v5, which requires changing them into the newer styled
API. There are currently 33 style files, which contain lots of styles not all of which are being used. (finos#1023) On top of that, the TS refactors make it harder to refactor the styles without getting type errors.
For the same amount of effort, we could remove our custom MUI styles entirely and use Tailwind instead, which will make our codebase cleaner and more manageable (at the cost of slightly more cluttered component code).
Describe the solution you'd like
Removing the MUI dependency (we could keep the icons), and recreate the styles with Tailwind. Since the app is not at all UI-heavy, it shouldn't take a lot of work other than getting the spacing and fonts right.
Describe alternatives you've considered
Upgrading MUI to v5 has been a pain, and it would not get rid of the extensive, and potentially unused styles in the app. Removing the unused styles would require going through each entry in each style file which is too much work.
Additional context
Related: finos#1023