From: Drew Adams <drew.adams@oracle.com>
To: Stefan Kangas <stefan@marxist.se>
Cc: "47047@debbugs.gnu.org" <47047@debbugs.gnu.org>,
Juri Linkov <juri@linkov.net>
Subject: bug#47047: [External] : Re: bug#47047: 28.0.50; 'help-key-binding' face's background sub-optimal
Date: Tue, 28 Sep 2021 20:26:38 +0000 [thread overview]
Message-ID: <SJ0PR10MB548820CF9AFF61C51D2A7453F3A89@SJ0PR10MB5488.namprd10.prod.outlook.com> (raw)
In-Reply-To: <CADwFkm=25F=-J6XjMf5hez4U_aLJ8sguYCH_C246vwmh+YsP5w@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 2549 bytes --]
> > You're free to delete light-blue-theme, if you like.
> > I don't use it, myself, and I won't miss it.
>
> Any other opinions about this part?
>
> I'm leaning towards just deleting it given the above.
Lean away. However, I don't see the point
of deleting a theme because you find that
the default appearance of a face you defined
looks, according to you, "out of place" with
that theme.
The bug of this thread is that the default
background of a (new) face is suboptimal. Is
there also some bug with the light-blue-theme?
You say that you think the new background you
chose for that new face looks out of place
with that theme. Maybe file a bug report for
that?
What's the relation between the background you
chose to use by default and the definition of
_any_ theme? Tail wagging dog? I'm pretty
sure that, for any theme, you can find a color
that you'll think looks out of place.
Emacs 28 isn't even released yet. You add a
face and then want to purge stuff that you
find "looks out of place" with your new face?
I won't try to stop you. But I find such a
purge a bit "out of place".
FWIW, I looked at a face with background
"gray90", which from the bug thread I gather
is the default you chose.
(defface foo '((t (:background "gray90")))
:group 'convenience)
I looked at it in my setup (which is about
the same as `light-blue-theme'), with
`list-faces-display'. Seems to show up OK.
(See attached screenshot.) Just how does
it look "out of place" to you?
If the point is about legibility/accessibility
then what counts is the value contrast between
foreground and background (nothing else really
counts). Assuming your face doesn't change the
foreground, the legibility of the face stands
on its own: the value difference between its
foreground (black?) and its background, gray90.
As for the face itself (its background)
standing out against a general background that
has a similar color value: yes, that hinders
accessibility. Someone with difficulty seeing
low value-contrast differences likely wouldn't
choose that theme anyway, or s?he might want to
customize some faces (including that new face).
An advantage of a pale (but not close to white)
background is the ability to easily see a range
of foreground colors, including very light and
very dark, but also mid-range (foregrounds and
backgrounds) if the hue is quite different. As
with any theme or color scheme (and any face,
including this new one), it's optional and
configurable.
[-- Attachment #2: throw-gray90-bg.png --]
[-- Type: image/png, Size: 319570 bytes --]
next prev parent reply other threads:[~2021-09-28 20:26 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-10 13:40 bug#47047: 28.0.50; 'help-key-binding' face's background sub-optimal Eli Zaretskii
2021-03-10 19:59 ` Juri Linkov
2021-03-10 20:06 ` Eli Zaretskii
2021-03-13 19:01 ` Stefan Kangas
2021-03-13 19:25 ` Stefan Kangas
2021-03-14 3:00 ` bug#47047: [External] : " Drew Adams
2021-03-14 3:39 ` Stefan Kangas
2021-09-28 17:46 ` Stefan Kangas
2021-09-28 20:26 ` Drew Adams [this message]
2021-09-28 20:54 ` bug#47047: [External] : " Stefan Kangas
2021-09-28 22:13 ` Drew Adams
2021-09-29 15:57 ` Lars Ingebrigtsen
2021-09-29 16:52 ` Stefan Kangas
2021-09-29 16:56 ` Lars Ingebrigtsen
2021-09-29 17:24 ` Stefan Kangas
2021-09-30 6:22 ` Lars Ingebrigtsen
2021-10-03 0:24 ` Stefan Kangas
2021-10-03 9:05 ` Lars Ingebrigtsen
2021-09-29 17:34 ` bug#47047: [External] : " Drew Adams
2021-09-29 19:18 ` Stefan Kangas
2021-09-29 19:50 ` Drew Adams
2021-09-29 21:17 ` Stefan Kangas
2021-09-29 22:21 ` Drew Adams
2021-09-29 23:21 ` Stefan Kangas
2021-09-30 1:59 ` Drew Adams
2021-09-28 17:31 ` Stefan Kangas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=SJ0PR10MB548820CF9AFF61C51D2A7453F3A89@SJ0PR10MB5488.namprd10.prod.outlook.com \
--to=drew.adams@oracle.com \
--cc=47047@debbugs.gnu.org \
--cc=juri@linkov.net \
--cc=stefan@marxist.se \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).