Skip to content

feat: add gbadwolf theme - #1

Open
gbrennon wants to merge 1 commit into
KovaMD:mainfrom
gbrennon:feat/gbadwolf-theme
Open

feat: add gbadwolf theme#1
gbrennon wants to merge 1 commit into
KovaMD:mainfrom
gbrennon:feat/gbadwolf-theme

Conversation

@gbrennon

@gbrennon gbrennon commented Jun 24, 2026

Copy link
Copy Markdown

Gruvbox is based in badwolf theme and this is my customization of it.
I already use it for my presentations and it could be good to share :)

Copilot AI review requested due to automatic review settings June 24, 2026 05:30

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@gbrennon
gbrennon force-pushed the feat/gbadwolf-theme branch from eb6f288 to 4a9e77d Compare June 24, 2026 05:33
@RDMillen RDMillen added the enhancement New feature or request label Jun 27, 2026
@RDMillen RDMillen self-assigned this Jun 27, 2026
@RDMillen

RDMillen commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

Hi @gbrennon, thanks for contributing this theme — the warm dark palette looks great for presentations!

A couple of accessibility issues worth addressing before we merge:

Chart colors are too close to each other
All 7 chart_colors are shades of orange and amber, so in any chart with multiple data series the colours are very hard to tell apart — especially for users with colour blindness. Could you spread them across more of the colour wheel? That said, you don't need to stray far from what makes this theme feel like itself — muted, desaturated versions of greens, blues, or purples can sit comfortably alongside the oranges without pulling away from the warm, earthy character of the palette.

accent, title_text, and chart_colors[0] are all the same colour (#d65d0e)
This means heading text, UI elements like links, and chart series 1 all look identical. On a slide with a chart it becomes hard to tell data from other elements. Something near-white for title_text (e.g. #f2e5bc) would keep the warm feel while making things easier to read.

@gbrennon

Copy link
Copy Markdown
Author

Hey there, @RDMillen !
How are u?

Got it!

Ill think in think in adjustments because of this issues related to accessibility!

I understand how important is to have accessibility bcs I was a wheelchair user for 3 but I'm walking again for 1 year and also have some eye issues.

Ill try to adjust this pr ;)
Also related to that last section in ur comment I have an explanation:

  • Those colors are mostly using #d65d0e because that is the main color of that theme.

Dont worry, I'll experiment other colors in the color wheel to try to adjust them and, if u could, i would be happy for more feedback after I update colors :)

@gbrennon

gbrennon commented Jun 28, 2026

Copy link
Copy Markdown
Author

@RDMillen could u tell me HOW to serve it so i can check my changes?
package seems to have disappeared from dnf(rn im in fedora)

@RDMillen

Copy link
Copy Markdown
Contributor

Hi @gbrennon,

Tested on Fedora, package is still present and happy. If it is not pulling the new version, try sudo dnf clean all && sudo dnf update

Ross

@gbrennon

gbrennon commented Jun 29, 2026

Copy link
Copy Markdown
Author

i thought that was already available using just sudo dnf install kova but following instructions i could install it.

how could i use my custom theme?
because its not merged yet and i want to verify it before updating this pr

@RDMillen

Copy link
Copy Markdown
Contributor

Hi @gbrennon,

To add to your own Kova install, just drop the yaml file into: ~/.config/kova/themes/ folder, then restart Kova and it should appear in your themes.

Assuming the repo is setup as per documentation, it should just work through sudo dnf install kova

Ross

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants