GNU bug report logs -
#33398
26.1, netbsd-8; Hi Lock Faces, Hi Green B, default setting suggestion
Previous Next
Reported by: Van L <van <at> scratch.space>
Date: Thu, 15 Nov 2018 12:59:02 UTC
Severity: wishlist
Found in version 26.1
Done: Juri Linkov <juri <at> linkov.net>
Bug is archived. No further changes may be made.
Full log
View this message in rfc822 format
[Message part 1 (text/plain, inline)]
Your bug report
#33398: 26.1, netbsd-8; Hi Lock Faces, Hi Green B, default setting suggestion
which was filed against the emacs package, has been closed.
The explanation is attached below, along with your original report.
If you require more details, please reply to 33398 <at> debbugs.gnu.org.
--
33398: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=33398
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
>> > (defface hi-redgold
>> > '((((min-colors 8)) (:background "red"))
>> > (t (:background "gold")))
>> > "Face for hi-lock mode."
>> > :group 'hi-lock-faces)
>> >
>> > I see no reason to use "red" on 16-color terminals, nor "orange" on
>> > color TTYs with at least 88 colors. The translation of "gold" should
>> > be good enough in those cases. Isn't it?
>>
>> I tried, and it used "red" even on 88-colors.
>
> I don't have access to such a terminal, but on a 16-color TTY "gold"
> displays as bright yellow, which is definitely not red. So I'm
> surprised you see red on a 88-color TTY.
Actually, I meant that I see red on a 88-color graphical display.
>> But given that LightSalmon is a good choice, I copied the same
>> definitions from other faces, substituted with new colors and they
>> look good on all environments:
>>
>> (defface hi-salmon
>> '((((min-colors 88) (background dark))
>> (:background "light salmon" :foreground "black"))
>> (((background dark)) (:background "red" :foreground "black"))
>> (((min-colors 88)) (:background "light salmon"))
>> (t (:background "red")))
>> "Face for hi-lock mode."
>> :group 'hi-lock-faces)
>>
>> (defface hi-aquamarine
>> '((((min-colors 88) (background dark))
>> (:background "aquamarine" :foreground "black"))
>> (((background dark)) (:background "blue" :foreground "black"))
>> (((min-colors 88)) (:background "aquamarine"))
>> (t (:background "blue")))
>> "Face for hi-lock mode."
>> :group 'hi-lock-faces)
>
> Thanks.
Installed to master and closed. It seems there is no need to update NEWS
because the users will see themselves two new colors added.
[Message part 3 (message/rfc822, inline)]
Hello,
This is a suggestion for a reasonable default fore/background color combo.
Run:
emacs -Q
See:
Customize Group: Hi Lock Faces
Hi Green B
I see:
foreground:green1
background:white (which I guess is a received setting from somewhere else)
For me, this is a very difficult combo to see.
I’d like to suggest the default to be for `Hi Green B':
foreground:darkcyan
This bug report was last modified 6 years and 156 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.