rougier / nano-modeline

GNU Emacs / N Λ N O Modeline
GNU General Public License v3.0
170 stars 29 forks source link

Invalid Face Error #57

Open vnckppl opened 1 year ago

vnckppl commented 1 year ago

When I try to run nano-modeline since the update to the modular version a week or so ago, I get the error 'Invalid Face':

Debugger entered--Lisp error: (error "Invalid face")
  internal-get-lisp-face-attribute(nil :foreground nil)
  face-attribute(nil :foreground nil nil)
  face-foreground(nil)
  (list t ':foreground (face-foreground (if (facep 'nano-faded) (progn 'nano-faded 'default))) ':background (face-background 'header-line nil t) ':family "Roboto Mono" ':box (cons ':line-width (cons 2 (cons ':color (cons (face-foreground 'default) '(:style none))))))
  (list (list t ':foreground (face-foreground (if (facep 'nano-faded) (progn 'nano-faded 'default))) ':background (face-background 'header-line nil t) ':family "Roboto Mono" ':box (cons ':line-width (cons 2 (cons ':color (cons (face-foreground 'default) '(:style none)))))))
  (custom-declare-face 'nano-modeline-button-inactive-face (list (list t ':foreground (face-foreground (if (facep 'nano-faded) (progn 'nano-faded 'default))) ':background (face-background 'header-line nil t) ':family "Roboto Mono" ':box (cons ':line-width (cons 2 (cons ':color (cons (face-foreground ...) '...)))))) "Inactive button face.")
  require(nano-modeline)
  eval-expression((require 'nano-modeline) nil nil 127)
  funcall-interactively(eval-expression (require 'nano-modeline) nil nil 127)
  command-execute(eval-expression)

To reproduce, start emacs with emacs -Q, and then run:

(add-to-list 'load-path "/your/path/to/nano-modeline-1.0.0")
(require 'nano-modeline)

I tried this on emacs 27.2, 29.0.60, 29.0.91, 30.0.50 and they all show this same error message.

larstvei commented 1 year ago

This looks like a duplicate of #50. Perhaps it deserves a minor version bump? I am just guessing that it is necessary for the change to reach ELPA.

rougier commented 1 year ago

Thanks. Just forgot to push to ELPA. Should be online in a few minutes.