-
Notifications
You must be signed in to change notification settings - Fork 5
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
MDX, Typescript, Mantine UI #145
base: master
Are you sure you want to change the base?
Conversation
I think overall I like the idea of typescript, but with it being strongly typed I worry about whoever might be picking up the webdev side of things once I'm gone. Javascript has a much lower barrier of entry. What do you think? Also, have you played with MUI at all? Great framework |
typescript for me is just a really good linter for javascript. its slightly more complicated, but I feel it's very easy to pick up and helps avoid common mistakes. |
4f200fb
to
b21a1fd
Compare
@sakgoyal Finally had a chance to review this. I'm really liking the updates! The format will need to be worked on, but once you knock that out I'd be happy to approve it. Resolved some of your merge conflicts so you can keep up the great work! |
@Forsyth-Creations I didn't realize you force pushed. my commits reverted your changes. I think you will need to redo those changes if they are required. |
Mantine also has built in dark mode