validkube icon indicating copy to clipboard operation
validkube copied to clipboard

Add themes dark and light

Open nirsht opened this issue 2 years ago • 8 comments

nirsht avatar Mar 28 '22 14:03 nirsht

Does the theme need to be toggled ON/OFF ?. I currently have a solution where it takes the system preference and sets the Dark or light mode. IF the theme needs to be toggled ON and OFF , Please let me know where the button should be placed. Attaching the light mode picture below. validkubeLightMode_

stevenovak123 avatar Sep 20 '22 14:09 stevenovak123

This is how it looks on 1920*1080 (attached below). For dark mode I have not made anychanges and kept the dark mode to be the original color scheme of validkube.

validKubeLightMode1920x1080

stevenovak123 avatar Sep 20 '22 14:09 stevenovak123

If the Above solution works. Please let me know . I will create a PR.

stevenovak123 avatar Sep 20 '22 14:09 stevenovak123

I think a toggle would make sense. Maybe three options: light/dark/system?

BrajBliss avatar Sep 20 '22 17:09 BrajBliss

@stevenovak123 toggle can be great, the pics looks fine. I wish that the toggle button will be with icons (like sun and moon) and not just ON/OFF.

nirsht avatar Sep 20 '22 18:09 nirsht

@stevenovak123 toggle can be great, the pics looks fine. I wish that the toggle button will be with icons (like sun and moon) and not just ON/OFF.

Alright I can do that and will submit a PR for it in a few days.

stevenovak123 avatar Sep 20 '22 20:09 stevenovak123

I think a toggle would make sense. Maybe three options: light/dark/system?

I am currently picking up the preference from the browser itself and making it light or dark. So having a system prefernce would not make sense to me. Will just turn it into a toggle for Light and Dark

stevenovak123 avatar Sep 20 '22 20:09 stevenovak123

I think a toggle would make sense. Maybe three options: light/dark/system?

I am currently picking up the preference from the browser itself and making it light or dark. So having a system prefernce would not make sense to me. Will just turn it into a toggle for Light and Dark

Sound good to use the browser itself. If any adjustments are needed in any component, feel free to change.

nirsht avatar Sep 21 '22 05:09 nirsht