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

Weird mode-line colors with nano-light #51

Open
maikol-solis opened this issue Feb 19, 2024 · 11 comments
Open

Weird mode-line colors with nano-light #51

maikol-solis opened this issue Feb 19, 2024 · 11 comments

Comments

@maikol-solis
Copy link
Contributor

Hi!

Today I tried nano-light since a long ago and I have these weird colors in the mode-line

image

The colors for mode-line
image

And for mode-line-inactive
image

In the code everything looks right, but I don't know if I missed something

`(mode-line ((,light (:foreground ,nano-light-background
:background ,nano-light-foreground
:box (:line-width 3
:color ,nano-light-foreground
:style nil)))

Thanks.

@maikol-solis
Copy link
Contributor Author

And nano-dark work as expected

image

@maikol-solis maikol-solis changed the title Weird mode-line colors Weird mode-line colors with nano-light Feb 19, 2024
@rougier
Copy link
Owner

rougier commented Feb 19, 2024

I think the theme is missing the mode-line-active that is a new face (I think). Can you check what's the face for mode-line-active (if it exists on your version) ?

@maikol-solis
Copy link
Contributor Author

Yes, mode-line-active inherits the face from mode-line.

image

@maikol-solis
Copy link
Contributor Author

Using custom-set-faces I added these two configs

 `(mode-line :inherit nano-subtle
    :background nil
    :foreground nil
    :box (:color ,nano-light-faded :style nil))
  `(mode-line-inactive
    :inherit nano-subtle-i
    :foreground nil
    :background nil
    :box nil)

An now the active mode-line looks like this

image

And the inactive one like this

image

It should be abstracted into the nano-theme structure, but I, personally, think it looks better.

WDYT?

@rougier
Copy link
Owner

rougier commented Feb 19, 2024

I looks nice! ActuallyI've tested the box in active mode before removing it because I found it a bit ditractng. A for the inactive mode, it's a bit too washed out.

@maikol-solis
Copy link
Contributor Author

Yes. You certainly have a better judgement about this than I do, and my initial idea could be improved, of course. If you have a better config, I can help you to test it.

@rougier
Copy link
Owner

rougier commented Feb 26, 2024

I'm actually using the box with nano-vertico mode (to show this is an input area). I get inspired by nyxt.

@maikol-solis
Copy link
Contributor Author

OK. I installed nano-vertico, and it improves a lot the whole UI. However, it has a similar issue with nano-dark

image

nano-vertico with the light theme works as expected

image

@rougier
Copy link
Owner

rougier commented Mar 18, 2024

Can you open an issue on nano-vertico?

@maikol-solis
Copy link
Contributor Author

Yes sure, no problem.

@dwla
Copy link

dwla commented Jul 26, 2024

maikol-solis’s config of mode-line and mode-line-inactive is intuitive. I find the box (as well as the darker background) in the minibuffer a little bit noisy, it’s more comfortable for me to make mode-line the same as the mode-line-inactive. Here is my config. BTW, nano-theme works well with doom-modeline!!!

(use-package nano-theme
  :custom-face
  (mode-line-buffer-id ((t (:foreground "black" :weight bold))))
  (mode-line-emphasis ((t (:foreground "black" :weight bold))))
  (doom-modeline-buffer-modified ((t (:foreground "black" :weight bold))))
  (doom-modeline-project-dir ((t (:foreground "black" :weight bold))))
  ;; mode-line coincides with mode-line-inactive
  (mode-line ((t (:background "#90A4AE" :foreground "#FFFFFF" :box (:line-width (3 . 3) :color "#90A4AE")))))
  (nano-popout ((t (:foreground "#B22222" :weight bold))))
  (nano-salient ((t (:foreground "#1E90FF" :weight semibold))))
  (vertical-border ((t (:foreground "black"))))
  ;; config for ess
  (doom-modeline-urgent ((t (:foreground "black" :weight bold))))
  (doom-modeline-warning ((t (:foreground "black" :weight bold))))
  (doom-modeline-info ((t (:foreground "black" :weight bold))))
  ;; config for eshell
  (eshell-prompt ((t (:foreground "#B03060"))))
  (dired-flagged ((t (:foreground "#FF3030" :weight bold))))
  :config
  (load-theme 'nano-light t))

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

No branches or pull requests

3 participants