Invalid face attribute :background nil

[Jousimies]

The Invalid face attribute :background nil error increase with mouse scroll
My config:

(leaf modus-themes
      :doc "https://protesilaos.com/modus-themes/"
      :load-path "~/.emacs.d/site-lisp/packages/modus-themes"
      :require t
      :commands (modus-themes-load-themes)
      :custom ((modus-themes-italic-constructs . t)
	       (modus-themes-bold-constructs . nil)
	       (modus-themes-region . '(bg-only no-extend))
	       (modus-themes-lang-checkers . '(background text-also straight-underline))
	       (modus-themes-links . '(faint))
	       (modus-themes-mode-line . '(accented borderless)))
      :init
      ;; Load the theme files before enabling a theme
      (modus-themes-load-themes)
      )

[protesilaos]

Hello @Jousimies and thanks for taking the time to report this.

I cannot reproduce this on my end. These are the steps I followed:

1. In a shell prompt type emacs -Q
2. Insert the following in the scratch buffer:

(require 'modus-themes "~/.emacs.d/site-lisp/packages/modus-themes/modus-themes.el") ; check the path
(setq modus-themes-italic-constructs t)
(setq modus-themes-bold-constructs nil)
(setq modus-themes-region '(bg-only no-extend))
(setq modus-themes-lang-checkers '(background text-also straight-underline))
(setq modus-themes-links '(faint))
(setq modus-themes-mode-line '(accented borderless))

(modus-themes-load-themes)

(load-theme 'modus-operandi t)

3. Run M-x eval-buffer
4. Now show the buffer with the echo area messages with C-h e.
5. Try to scroll up and down.

Can you try those steps and let me know if you get the same results as I do?

If you do, then there must be something wrong with another package that you use and how it expects some background value that does not exist. Maybe there still is something in the themes, but we need to determine what produces the error.

[protesilaos]

I think I found the problem on the side of the themes. I pushed the changes. Though there still may be another package involved.

[Jousimies]

@protesilaos Thank you for your help. I found the problem related to the centaur-tabs package.

[protesilaos]

Very well! If you can pinpoint it, then we should report it to the centaur-tabs maintainer so that other users do not experience similar problems.

Also, I would be curious to know what it is, just so I can learn what to expect next time something similar happens.

[jeremyf]

@Jousimies do you have the solution for this; I'm seeing it in my build as well.

[protesilaos]

@jeremyf I am trying to fix the problem upstream: ema2159/centaur-tabs#179. Basically, the way centaur-tabs' faces are designed right now prevent us from using :inherit (which is something that themes should be able to do).

Relevant issue in the GitLab mirror: https://gitlab.com/protesilaos/modus-themes/-/issues/288