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

Colors don't work in emacs-nox #709

Open
cryslith opened this issue Oct 9, 2023 · 1 comment
Open

Colors don't work in emacs-nox #709

cryslith opened this issue Oct 9, 2023 · 1 comment

Comments

@cryslith
Copy link

cryslith commented Oct 9, 2023

This is because proof-general's proof-face-specs function defines its color classes like:

((((type x) (class color) (background light)) (:background "#eaf8ff" :extend t)) (((type x) (class color) (background dark)) (:background "darkslateblue" :extend t)) (((type mswindows) (class color) (background light)) (:background "#eaf8ff" :extend t)) (((type mswindows) (class color) (background dark)) (:background "darkslateblue" :extend t)) (((type w32) (class color) (background light)) (:background "#eaf8ff" :extend t)) (((type w32) (class color) (background dark)) (:background "darkslateblue" :extend t)) (((type gtk) (class color) (background light)) (:background "#eaf8ff" :extend t)) (((type gtk) (class color) (background dark)) (:background "darkslateblue" :extend t)) (((type mac) (class color) (background light)) (:background "#eaf8ff" :extend t)) (((type mac) (class color) (background dark)) (:background "darkslateblue" :extend t)) (((type carbon) (class color) (background light)) (:background "#eaf8ff" :extend t)) (((type carbon) (class color) (background dark)) (:background "darkslateblue" :extend t)) (((type ns) (class color) (background light)) (:background "#eaf8ff" :extend t)) (((type ns) (class color) (background dark)) (:background "darkslateblue" :extend t)) (((type x-toolkit) (class color) (background light)) (:background "#eaf8ff" :extend t)) (((type x-toolkit) (class color) (background dark)) (:background "darkslateblue" :extend t)) (t (:underline t :extend t)))

by enumerating over the list of all possible display types.
This fails to handle terminals even though many terminals support color.
It would be better to drop the type restrictions and define faces based on which graphical features (like color) the display supports.

@monnier
Copy link
Contributor

monnier commented Oct 9, 2023

+1

Patch welcome

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

No branches or pull requests

2 participants