Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor project styling to use TailwindCSS #205

Open
digitdustin opened this issue Jan 30, 2023 · 1 comment
Open

Refactor project styling to use TailwindCSS #205

digitdustin opened this issue Jan 30, 2023 · 1 comment

Comments

@digitdustin
Copy link
Contributor

Upgrade styling to use TailwindCSS instead of MUI to:

  • Improve developer experience
  • Decrease styling bias
  • Decrease frustration in avoiding overriding styles
  • Allow for upgrade to NextJS13 (MUI currently not supported with server components)
@marcgabe15
Copy link
Contributor

marcgabe15 commented Feb 4, 2023

Note: NextJS13 app directory uses server components by default and if we want client components (which our app needs) we'll need to put "use client" on every single file. Our current app right now is hosted just a static files hosted over a cdn so by default we still can't use app directory unless we want to move towards running this on an actual server. Next.js is tightly coupled with Vercel's system so its super opinionated on using SSR.

Some things to think about:

  • What is the value add for us to move to SSR vs using CSR right now?

Some advantages I can think of right now:

  • Vercel is free! (will it be free forever?)
  • SEO/Page ranking

Disadvantages:

  • It's easier to set up just some static html/css/js files rather than fully loaded ssr
  • TTI (time to interactive) is slower on SSR which in our case our app is more meant to be interactive than read only

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants