Fix dark mode colors#552
Conversation
PR ctrliq#541 changed page/card/table/toolbar backgrounds from their correct values (#000 for pages, #1a1e25 for cards) to a uniform #13161b, making cards invisible against the page background. Restore the original color hierarchy and keep the valid new additions (status label colors, menu-toggle, ace tag/markup, thead border, legend/tooltip, wizard sections).
|
@cigamit Check what I did. I added CtrllQ green on the links, buttons, etc. on both light and dark mode themes under PF6. So the brand "green" is everywhere and now even light mode has rounded buttons, etc. Your version of the dark theme dropped certain visual elements and made buttons green where buttons were still disabled (e.g. the search icon was green even before writing text to search). I think this PR's PF6 light & dark css are the correct ones and they look modern and branded :P |
|
A few colleagues and I like the version with the black background more - we think it feels more polished. But of course, it's your product and your call ^^ |
|
Ya, my UI team wants me to pull in their entire UI suite, and change the icons and everything, but I have pushed back as I don't like the look it gives. What we have now was the compromise. |


Summary
Test plan
html.pf-v6-theme-darkselectors)