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] Warning: setting attribute ‘:background’ of face ‘font-lock-comment-face’: nil value is invalid, use ‘unspecified’ instead. #415

Closed
3 tasks done
CsBigDataHub opened this issue Jul 31, 2023 · 3 comments
Labels

Comments

@CsBigDataHub
Copy link
Contributor

Thank you for the bug report

  • I am using the latest version of Centaur Emacs.
  • I checked my private configurations in custom.el and custom-post.el.
  • I checked FAQ.

Bug description

Warning: setting attribute ‘:background’ of face ‘font-lock-comment-face’: nil value is invalid, use ‘unspecified’ instead.

Getting this at start up. This is from main branch after feature/modern-comp merge.

Steps to reproduce

Check messages buffer after start up

Expected behavior

No warnings

OS

MacOS

Emacs Version

gccemacs-29

Error callstack

No response

Anything else

No response

@seagle0128
Copy link
Owner

See doomemacs/themes#793.

@CsBigDataHub
Copy link
Contributor Author

CsBigDataHub commented Aug 1, 2023

Thanks @seagle0128 .

Question on system themes.

in custom-post.el

(centaur-system-themes '((light . modus-operandi)
                             (dark  . modus-vivendi)))

and

in custom.el

(setq centaur-theme 'system) 

Customizing this variable does not allow me to use modus-themes. What is the right way to configure modus themes in custom-post.el ?

@CsBigDataHub
Copy link
Contributor Author

CsBigDataHub commented Aug 2, 2023

Able to figure it out.

(custom-set-variables
 '(centaur-system-themes '((light . modus-operandi) (dark . modus-vivendi)))
 ;; '(modus-themes-common-palette-overrides modus-themes-preset-overrides-intense)
 '(modus-themes-completions '((t . (extrabold intense underline background))))
 '(modus-themes-headings '((1 . (background overline variable-pitch 1.5))
                           (2 . (overline rainbow 1.3))
                           (3 . (overline 1.1))
                           (t . (monochrome))))
 '(modus-themes-italic-constructs t)
 '(modus-themes-mixed-fonts t)
 '(modus-themes-org-blocks 'tinted-background)
 '(modus-themes-prompts '(intense background bold))
 '(modus-themes-variable-pitch-ui t))

But when I set this variable below, it is conflicting with solaire-mode and saying the solaire-mode is not loaded.

'(modus-themes-common-palette-overrides modus-themes-preset-overrides-intense)

How do I elegantly enable this variable with out conflicting with solaire-mode.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants