search

sfbrigade / sf-lifeline

GNU Affero General Public License v3.0
7 stars 4 forks source link

Mantine Integration #16

Closed rooiss closed 10 months ago

rooiss commented 10 months ago

added all mantine packages, postcss in devDependencies, and wrapped app in mantine provider

rooiss commented 10 months ago

Do we really want to add ALL the mantine packages to start vs bringing them in as needed/used?

Also, do we need to add the postcss.config.cjs file as described in the docs: https://mantine.dev/guides/vite/#postcss-setup

?

@francisli Yeah I was thinking that as well, I'll remove all of them except for the hooks and core. Makes more sense to bring things in as needed.

After reading more of what the postcss configuration is for, looks like we don't need it as of now so i'll remove both of those.

Can you review again and let me know if everything looks good?

rooiss commented 10 months ago

I see that the recharts module is still in dependencies... that's for creating chart visualizations, we can probably also remove that until needed...?

@francisli Oops, didn't see that one there, went ahead and removed that and pulled dev into this branch