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.

Sign up for GitHub

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

Jump to bottom

Warning: nil value is invalid, use ‘unspecified’ instead #810

Closed
jabotz opened this issue Dec 2, 2023 · 3 comments
Closed

Warning: nil value is invalid, use ‘unspecified’ instead #810

jabotz opened this issue Dec 2, 2023 · 3 comments
Labels
duplicate This issue or pull request already exists

Comments

@jabotz
Copy link

jabotz commented Dec 2, 2023

Theme(s)

doom-nord-light

What were you expecting?

No warnings

What actually happened?

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

Steps to reproduce

  1. Run emacs 29.1
  2. Switch to doom-nord-light theme
  3. This can be fixed by changing lines 135 and 149 in doom-nord-light-theme.el

Operating system

Windows 10, 11

Emacs version

Emacs 29.1

Installed commit of doom-themes

4aee1f5
@jabotz jabotz added is:bug Something isn't working as intended needs-triage Issue hasn't been assessed yet labels Dec 2, 2023
@jabotz
Copy link
Author

jabotz commented Dec 2, 2023

This is not a duplicate of https://github.com/doomemacs/themes/issues/793, but a similar further issue.

@hlissner hlissner added duplicate This issue or pull request already exists and removed is:bug Something isn't working as intended needs-triage Issue hasn't been assessed yet labels Dec 2, 2023
@hlissner
Copy link
Member

hlissner commented Dec 2, 2023

This is a duplicate of #793, which covers all nil->unspecified warnings of this kind, across all themes. Not sure what's supposed to make this different. In any case, this should be fixed in fe4a9b4.

@hlissner hlissner closed this as not planned Won't fix, can't repro, duplicate, stale Dec 2, 2023
@jabotz
Copy link
Author

jabotz commented Dec 2, 2023

Sorry, I've now realised that melpa doesn't have the most recent commit. Will wait for its upload.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
duplicate This issue or pull request already exists
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hlissner @jabotz