rougier / nano-modeline

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

Cannot get the simpler nano-modeline to work #55

Closed yakhni closed 1 year ago

yakhni commented 1 year ago

Hi, I am trying to do the following:

(use-package! nano-modeline
  :config
  (nano-modeline-text-mode t))

However, I end up with the following backtrace:

Debugger entered--Lisp error: (wrong-type-argument listp nano-modeline--empty-face)
  face-remap-set-base(header-line nano-modeline--empty-face)
  nano-modeline-header(((nano-modeline-buffer-status) " " (nano-modeline-buffer-name) " " (nano-modeline-git-info)) ((nano-modeline-cursor-position) (nano-modeline-window-dedicated)) t)
  funcall(nano-modeline-header ((nano-modeline-buffer-status) " " (nano-modeline-buffer-name) " " (nano-modeline-git-info)) ((nano-modeline-cursor-position) (nano-modeline-window-dedicated)) t)
  nano-modeline-text-mode(t)

I'm using emacs 28.2, doom emacs. I've been using nano-modeline and nano-theme for quite a bit, but I cannot get nano-modeline to run with the new API. Please advise.

rougier commented 1 year ago

Can you try:

(require 'nano-modeline)
(nano-modeline-text-mode t)
coldfish commented 1 year ago

Still getting the same error like @yakhni

(wrong-type-argument listp nano-modeline--empty-face)

rougier commented 1 year ago

Might be linked with a fix I pushed a few minutes ago on ELPA. Coudl you try with the github version (I think ELPA still not updated) ?

coldfish commented 1 year ago

I'm using the github version directly and see the last commit in my repo list:

Author: Nicolas P. Rougier
Date:   Mon Jun 19 16:12:53 2023 +0200

    Minor bugfix for ELPA
rougier commented 1 year ago

What is your version of emacs?

coldfish commented 1 year ago

What is your version of emacs?

28.2

rougier commented 1 year ago

Oh, is it the same issue #59 ?

yakhni commented 1 year ago

I am closing this issue since upgrading to emacs version 29.1 resolves this issue.