bazarr icon indicating copy to clipboard operation
bazarr copied to clipboard

Upgrade mantine to v7.x

Open anderson-oki opened this issue 2 months ago • 1 comments

Description

The main change on the Mantine V7 is that they removed the usage of Emotion and now all the customizations diretly from CSS in JS is not recommended, instead we should either use CSS Modules or start using another dependency such as Vannila-Extract. We can discuss later on adding Vannila-Extract, however i would recommend to keep focusing on Mantine and their recommendations, otherwise future updates might be even more terrifying.

Changes

  • Added SASS configuration for vite
  • Moved all CSS in JS to SCSS Modules
  • MultiSelectorProps was removed from Matine so replaced with TagsInput temporary until we develop our own solution
  • The plain table, thead, tr and td all have its own components from Mantine.

Next Steps

  • The old MultiSelectorProps was a little bit more fancy as it displayed some advices instead of just the default behavior of pressing Enter which is probably more intuitive. The current TagsInput keeps the same functionality but would be nice to have our own custom MultiSelector. To minimize the changes i prefer to do in a different PR once this one is merged.

anderson-oki avatar Apr 22 '24 14:04 anderson-oki

@morpheus65535 The PR is ready, but would be great if i can have some help for review and QA before merging.

anderson-oki avatar Apr 23 '24 02:04 anderson-oki

@morpheus65535 I applied some changes to make it closer to what we have now without having to hack and get screwed during the next updates.

anderson-oki avatar Apr 30 '24 06:04 anderson-oki

Let me do some extra refactorings here, also there is one bug on the sync subtitle modal. Converting it to Draft.

anderson-oki avatar Apr 30 '24 11:04 anderson-oki