Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug: Some colors are very hard to see #14

Closed
3 tasks done
lespea opened this issue Feb 13, 2024 · 4 comments · Fixed by #15
Closed
3 tasks done

bug: Some colors are very hard to see #14

lespea opened this issue Feb 13, 2024 · 4 comments · Fixed by #15
Assignees
Labels
bug Something isn't working

Comments

@lespea
Copy link

lespea commented Feb 13, 2024

Did you check docs and existing issues?

  • I have read all the pastelnight.nvim docs
  • I have searched the existing issues of pastelnight.nvim
  • I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)

0.9.5

Operating system/version

Linux 6.7.4

Describe the bug

Here are two examples where it's very difficult to read text.

https://imgur.com/a/DjyVQoy

The top example is in lualine and the second is in htop (using the kitty/fish theme files). I also noticed the separator between vim windows is now invisible? The colors theme to line up with the pictures so I'm pretty sure they're being applied correctly.

Steps To Reproduce

  1. Add theme to neovim
  2. Add lualine
  3. In setup opts specify theme = 'pastelnight'
  4. Look at bottom text in lualine some of it is very dark

-- or

  1. Add colorscheme to kitty
  2. Open htop
  3. Look at cpu usage text

Expected Behavior

Text stands out more so it's easier to read

Repro

-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .reprpastelnight
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "pauchiner/pastelnight.nvim",
  "pauchiner/pastelnight.nvim",
  -- add any other plugins here
  "nvim-lualine/lualine.nvim",
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("pastelnight")
-- add anything else here
-- You need to set the pastelnight theme in lualine
@lespea lespea added the bug Something isn't working label Feb 13, 2024
@pauchiner pauchiner self-assigned this Feb 13, 2024
@pauchiner
Copy link
Owner

I just merged a PR increasing the contrasts for the lualine and lightline plugins. I also updated all the terminal extras to increase the color contrast of the bright black the color causing this weird contrast in htop.

Anyway, the relative low contrast of the theme, is a design choice, but I think it would be great to have a colorscheme variant with higher contrast for those who need it. I already opened a issue for this #17.

@pauchiner
Copy link
Owner

The window separator highlight is already set, but is only visible on the vertical windows:

Window Separator Example

I'm currently researching how to solve this.

@lespea
Copy link
Author

lespea commented Feb 13, 2024

Looks great, thanks!

Re windows: interesting yeah I do see the window separator now it turns out that for kyazdani42/nvim-tree.lua that separator is invisible which is weird I thought that was just a normal window? Maybe it's related to why it's invisible on vertical windows but that's just a guess.

@pauchiner
Copy link
Owner

@lespea In fact, the nvim-tree.lua plugin has it's own highlight for the window border, is independent of the rest of the workspace.

The really issue might be with some conflict between statusline plugins, like lualine with the theme base config, which causes the horizontal window border to disappear.

I'm going to close this issue, if you encounter more problems with the theme, feel free to open another.

@pauchiner pauchiner linked a pull request Feb 13, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants