-
-
Notifications
You must be signed in to change notification settings - Fork 78
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
Don't have any scoped CSS #375
Comments
@linkdotnet I would like to work on this issue. can you assign it to me? |
Love it - thanks and done ;) also don’t feel pressured to do all the work. I am fine if we split this into multiple PRs. |
Hey @otutukingsley I removed the assignment for now. If you work on this, let me know and I add you again |
Hello there, how is going this task? I am interested in providing more support to the project and it seems this one at my reach and for learning into front-end too. |
If you want, feel free to pick it up and ask me any question regarding it. My idea would not only be removing any scoped CSS and move it to the root CSS, but also use more bootstrap utility classes when applicable. I started my journey without much knowledge of bootstrap itself, so I created way too much custom CSS where it wasn't necessary. So probably I would do it in 3 steps:
|
Thanks for the info and steps will write when I get some progress |
I have a stronger opinion, that we should go into the direction of something like tailwindcss that makes a lot of things easier. I tried it out in another Blazor project, and with v4, it was a pleasant experience. |
While scoped CSS is nice, it also makes certain things like creating critical CSS more complicated.
Therefore all scoped CSS should either:
The text was updated successfully, but these errors were encountered: