search

freeCodeCamp / Developer_Quiz_Site

This is a quiz site filled with over 1500+ questions on programming.
https://developerquiz.org
BSD 3-Clause "New" or "Revised" License
267 stars 360 forks source link

[imporvement] - consider to remove react-bootstrap package #1211

Closed shootermv closed 1 week ago

shootermv commented 3 weeks ago

What bug did you find in the codebase?

currently the project uses react-bootstrap package for styles. as far as i can see - there is no big need for bootstrap styles in the project... (only the modal actually makes use of bootstrap package - but it can be implemented in other ways ) it can be nice to move to tailwind styling as it in the main project

jdwilkin4 commented 3 weeks ago

it can be nice to move to tailwind styling as it in the main project

I don't see a need to move to tailwind. Using regular CSS for this project is fine. The main repo uses tailwind but it is on a completely different scale and manages way more components and styles then this one.

shootermv commented 3 weeks ago

It is makes sense that tailwind is overkill

On Fri, Nov 1, 2024, 13:47 Jessica Wilkins @.***> wrote:

it can be nice to move to tailwind styling as it in the main project

I don't see a need to move to tailwind. Using regular CSS for this project is fine. The main repo uses tailwind but it is on a completely different scale and manages way more components and styles then this one.

— Reply to this email directly, view it on GitHub https://github.com/freeCodeCamp/Developer_Quiz_Site/issues/1211#issuecomment-2451746190, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKGMHSWTCFXLVI6GT7JDETZ6NS3PAVCNFSM6AAAAABRAAOC7KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJRG42DMMJZGA . You are receiving this because you authored the thread.Message ID: @.***>