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

Update: Use Tailwind CSS instead of separate CSS files #46

Open
Sambit-Mondal opened this issue Oct 3, 2024 · 6 comments · May be fixed by #168
Open

Update: Use Tailwind CSS instead of separate CSS files #46

Sambit-Mondal opened this issue Oct 3, 2024 · 6 comments · May be fixed by #168
Assignees

Comments

@Sambit-Mondal
Copy link
Contributor

Reasons:

  • If we use Tailwind CSS instead of separate CSS files, it'd make the app more light-weight, and would increase the speed of the application.
  • It'd reduce the chances of error of other contributors contributing to this project, as they don't have to write the CSS from scratch.
  • It'd be easy to understand too, as we don't have to find each element separately in the respective CSS files through their respective classnames.
  • Since it is built on MERN Stack, having Tailwind CSS as the styling framework would be an industry level enhancement in this project.
@vishalverma9572
Copy link
Owner

@Sambit-Mondal , can you explain how you're planning to work on this? Are you going to restyle all the components similarly, or do you have a different approach?

@Sambit-Mondal
Copy link
Contributor Author

@vishalverma9572 Yeah, I was thinking to restyle all the components similarly, as they are, but using Tailwind CSS. Will it be okay?

@vishalverma9572
Copy link
Owner

@Sambit-Mondal , yes, work on the components one by one. If there are no issues, create a PR, and after that, you can move on to other components. We follow a specific styling flow where we create a CSS file with the same name as the component, and for each style, we add .parent_div in that component's classes or IDs. Please keep this in mind.

@Sambit-Mondal
Copy link
Contributor Author

@vishalverma9572 Okay, I'll take care of it.

@vishalverma9572
Copy link
Owner

@Sambit-Mondal , any updates?

@Sambit-Mondal
Copy link
Contributor Author

@vishalverma9572 Actually, I didn't work in the previous week, as I was out of station for Durga Puja. I'll start working again by tomorrow, and will complete it and create a PR by the end of this week.

Sorry for the delay and thanks for your patience! 😊

@Sambit-Mondal Sambit-Mondal linked a pull request Oct 26, 2024 that will close this issue
13 tasks
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

Successfully merging a pull request may close this issue.

2 participants