Skip to content

fix: apply bug fixes from original repo PRs - #14

Merged
rianjs merged 4 commits into
mainfrom
fix/apply-bug-fixes
Jan 24, 2026
Merged

fix: apply bug fixes from original repo PRs#14
rianjs merged 4 commits into
mainfrom
fix/apply-bug-fixes

Conversation

@rianjs

@rianjs rianjs commented Jan 24, 2026

Copy link
Copy Markdown

Summary

Applies 4 bug fixes from dexpota/kitty-themes PRs that fix usability issues in existing themes.

Fixes Applied

Theme Problem Fix Source
OneDark color5/color13 don't match spec #be5046 → #c678dd dexpota#64
gruvbox_dark color11 out of sync with upstream #fabd2d → #fabd2f dexpota#43
Solarized_Light Selection nearly invisible Improved contrast dexpota#42
Solarized_Dark Selection nearly invisible Improved contrast dexpota#41

Related Issues

These fixes address:

All themes validated with ./scripts/validate-themes.sh.

Closes #13

color5 and color13 now match the onedark.vim color reference spec.

Ref dexpota#64
color11 was #fabd2d, should be #fabd2f per morhetz/gruvbox.

Ref dexpota#43
Selection was nearly invisible. Updated selection_background and
selection_foreground for better visibility.

Ref dexpota#42
Selection was nearly invisible. Updated selection_background and
selection_foreground for better visibility.

Ref dexpota#41
@rianjs
rianjs merged commit 776061c into main Jan 24, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: Apply bug fixes from original repo PRs

1 participant