Metronic 8.1 + Bootstrap 5.2 + CSS Variables
Hey folks! Question for you:
Now that Bootstrap 5.2 is out and has much more extensive use of CSS Variables, what is the reasoning behind having KeenThemes CSS Variables that override those values, vs just setting the Bootstrap built-in ones wherever possible?
Thanks for your feedback.
We did override the Bootstrap css variables to support both light/dark mode within the same css file. Bootstrap 5.3 should come with dark mode support and then we will switch back to the Bootstrap's native dark mode support. Till then, we provide our own dark mode solution.
Hey Sean! Thanks for getting back to me so quickly.
I'm actually referring to situations like this:
color: var(--kt-text-primary) !important;
But yes, inside the light vs dark theming, there are also a bunch of places where you've introduced your own variables and override the in later classes, where simply setting the built-in values appears as if it would have been sufficient.
It adds a significant amount of bloat to the CSS file that doesn't seem necessary.
No problem at all. Good point. We will revise that part and try to reduce such code duplications. Appreciate your suggestion.