all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
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 --]

  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

* 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 external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.