sxyazi / yazi

💥 Blazing fast terminal file manager written in Rust, based on async I/O.
https://yazi-rs.github.io
MIT License
14.63k stars 331 forks source link

default theme is not friendly for light terminal background #149

Closed tkkcc closed 11 months ago

tkkcc commented 1 year ago

yazi 0.1.4 on archlinux, no custom config alacritty terminal with light background open yazi and type g, the keymap prompt text is unrecognizable: image

sxyazi commented 1 year ago

Have you made a light theme for Yazi?

tkkcc commented 1 year ago

no yazi config, but I have following $LS_COLORS, is it related?

rs=0:di=01;34:ln=01;36:mh=00:pi=01;33:so=01;35:do=01;35:bd=01;33;01:cd=01;33;01:or=01;31;01:mi=00:su=01;41:sg=01;43:ca=01;41:tw=01;42:ow=0:st=01;44:ex=01;32:*.tar=01;31:*.tgz=01;31:*.arc=01;31:*.arj=01;31:*.taz=01;31:*.lha=01;31:*.lz4=01;31:*.lzh=01;31:*.lzma=01;31:*.tlz=01;31:*.txz=01;31:*.tzo=01;31:*.t7z=01;31:*.zip=01;31:*.z=01;31:*.Z=01;31:*.dz=01;31:*.gz=01;31:*.lrz=01;31:*.lz=01;31:*.lzo=01;31:*.xz=01;31:*.zst=01;31:*.tzst=01;31:*.bz2=01;31:*.bz=01;31:*.tbz=01;31:*.tbz2=01;31:*.tz=01;31:*.deb=01;31:*.rpm=01;31:*.jar=01;31:*.war=01;31:*.ear=01;31:*.sar=01;31:*.rar=01;31:*.alz=01;31:*.ace=01;31:*.zoo=01;31:*.cpio=01;31:*.7z=01;31:*.rz=01;31:*.cab=01;31:*.wim=01;31:*.swm=01;31:*.dwm=01;31:*.esd=01;31:*.jpg=01;35:*.jpeg=01;35:*.mjpg=01;35:*.mjpeg=01;35:*.gif=01;35:*.bmp=01;35:*.pbm=01;35:*.pgm=01;35:*.ppm=01;35:*.tga=01;35:*.xbm=01;35:*.xpm=01;35:*.tif=01;35:*.tiff=01;35:*.png=01;35:*.svg=01;35:*.svgz=01;35:*.mng=01;35:*.pcx=01;35:*.mov=01;35:*.mpg=01;35:*.mpeg=01;35:*.m2v=01;35:*.mkv=01;35:*.webm=01;35:*.ogm=01;35:*.mp4=01;35:*.m4v=01;35:*.mp4v=01;35:*.vob=01;35:*.qt=01;35:*.nuv=01;35:*.wmv=01;35:*.asf=01;35:*.rm=01;35:*.rmvb=01;35:*.flc=01;35:*.avi=01;35:*.fli=01;35:*.flv=01;35:*.gl=01;35:*.dl=01;35:*.xcf=01;35:*.xwd=01;35:*.yuv=01;35:*.cgm=01;35:*.emf=01;35:*.ogv=01;35:*.ogx=01;35:*.aac=00;36:*.au=00;36:*.flac=00;36:*.m4a=00;36:*.mid=00;36:*.midi=00;36:*.mka=00;36:*.mp3=00;36:*.mpc=00;36:*.ogg=00;36:*.ra=00;36:*.wav=00;36:*.oga=00;36:*.opus=00;36:*.spx=00;36:*.xspf=00;36:
sxyazi commented 1 year ago

This is related to the theme system, and I am currently in the middle of refactoring the theme system.

I will address this once I have finished it.

asim215 commented 1 year ago

I have following with gruvbox light alacritty and after tweaking config. 20230914_18h41m04s_grim

sxyazi commented 11 months ago

Done by https://github.com/sxyazi/yazi/pull/221, please give it a try to see if it works for you!

magnetophon commented 11 months ago

Thanks for your work on this, @Eric-Song-Nop and @sxyazi! When renaming, I still get unreadable text:

image

Eric-Song-Nop commented 11 months ago

Thanks for your work on this, @Eric-Song-Nop and @sxyazi! When renaming, I still get unreadable text:

@magnetophon Can you provide me with the your theme file? I guess I know what's going wrong here, maybe you need to also change the syntect_theme entry of preview in theme.toml to a light theme also

sxyazi commented 11 months ago

Hi @magnetophon, I made a new PR to fix it, https://github.com/sxyazi/yazi/pull/264, please give it a try.

magnetophon commented 11 months ago

@magnetophon Can you provide me with the your theme file?

I was testing without any config for yazi.

Hi @magnetophon, I made a new PR to fix it, #264, please give it a try.

Compiling as we speak.

magnetophon commented 11 months ago

Yes, all fixed for me in https://github.com/sxyazi/yazi/pull/264. Thanks!

magnetophon commented 10 months ago

There are still two cases where I get unreadable text. 1) When I interactive-cd: image

2) In the help screen: image

This is with the default theme.

sxyazi commented 10 months ago

Hi @magnetophon, what terminal theme are you using? My iTerm2 with Tango Light looks like:

CleanShot 2023-11-07 at 19 22 12

By using the default theme config:

[completion]
border   = { fg = "blue" }
active   = { bg = "darkgray" }
inactive = {}

# Icons
icon_file    = ""
icon_folder  = ""
icon_command = ""
magnetophon commented 10 months ago

I'm using wezterm. When I try with alacritty, it's all readable.

I guess that means it's a bug in the theme I'm using for wezterm.

Wezterm has two implementations of Solarized light. When I use the other one, the issue goes away.

Thanks for pointing me in the right direction!

github-actions[bot] commented 9 months ago

I'm going to lock this issue because it has been closed for 30 days. ⏳ This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.