kovidgoyal / kitty-themes

Themes for the kitty terminal emulator
GNU General Public License v3.0
417 stars 83 forks source link

Fix Modus and Kaolin themes visibility for kittens #86

Closed alternateved closed 1 year ago

alternateved commented 1 year ago

Proper fix for https://github.com/kovidgoyal/kitty-themes/issues/68 and https://github.com/kovidgoyal/kitty/issues/6320 where color15 wasn't visible for select_tab and unicode_input kittens. This shouldn't break TUI applications as https://github.com/kovidgoyal/kitty-themes/pull/85 did.

jian-lin commented 1 year ago

When Modus Operandi theme is enabled both in kitty and Emacs, it does not look good:

  1. by default the comments are invisible
  2. only highlighted comments are visible
  3. background color does not look good image

Before this PR, it looks good with the same config: image

alternateved commented 1 year ago

@jian-lin Sorry, I cannot reproduce it using those themes. I've refreshed themes cache (with kitty +kitten themes --cache-age 0) and it works properly on my side with modus-themes from MELPA.

jian-lin commented 1 year ago

I find that the issue I mention in https://github.com/kovidgoyal/kitty-themes/pull/86#issuecomment-1573263223 is a bit tricky to reproduce.

My Emacs version is GNU Emacs 28.2 (build 1, x86_64-pc-linux-gnu, X toolkit, cairo version 1.16.0, Xaw3d scroll bars) and the version of modus-themes is 20230530.627 from MELPA.

My init.el file is

(eval-when-compile
  (require 'use-package))

(use-package modus-themes
  :init
  (load-theme 'modus-operandi t)
  :custom
  (modus-themes-italic-constructs t))

This issue does not occur if I open kitty and then run one of the following commands

This issue does occur if I open kitty and then run one of the following commands

Looks like it is caused by environment variables:

jian-lin commented 1 year ago

After checking the diff of environment variables before and after ssh localhost, I find that COLORTERM=truecolor is the root cause: starting Emacs without it causes the issue in https://github.com/kovidgoyal/kitty-themes/pull/86#issuecomment-1573263223.

So that issue is likely a config issue on my side.

Thanks!