From: Drew Adams <drew.adams@oracle.com>
To: Lars Ingebrigtsen <larsi@gnus.org>
Cc: 15427@debbugs.gnu.org
Subject: bug#15427: 24.3.50; `customize-face' in Customize, show `Background' attribute using background
Date: Wed, 14 Aug 2019 21:44:44 -0700 (PDT) [thread overview]
Message-ID: <d87f3b7b-c067-498e-bda8-a05565780f69@default> (raw)
In-Reply-To: <87k1bf3xsi.fsf@mouse.gnus.org>
> > emacs -Q
> > M-x customize-face lazy-highlight
> >
> > Attribute `Background' has its own `sample', which shows the color.
> > But it shows the color as a foreground (text). Show it as the
> > background, instead.
> >
> > This is independent of the overall `sample' for the face as a whole.
> > It's just more helpful to show the background color choice as a
> > background.
>
> There's a whole bunch of things that can have a colour when customising
> a face -- like stipple background/foreground, overline, etc. I don't
> know whether it makes sense to special-case just the (main) background
> colour... but perhaps that does make sense, because it's a pretty
> common thing to customise. And showing the colour as a background
> colour would give a more correct impressions.
>
> But might be confusing if that one thing behaved differently than the
> rest.
>
> Anybody have an opinion here?
It's _not_ about one thing behaving differently.
We show an overline, underline, foreground, etc.
attribute using the foreground, because they are,
well, in the foreground.
Attribute background is not. Attribute background
should, well, show the color sample as background,
not as foreground. (I would think that this is
obviously TRT to do.)
next prev parent reply other threads:[~2019-08-15 4:44 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-20 17:04 bug#15427: 24.3.50; `customize-face' in Customize, show `Background' attribute using background Drew Adams
2019-08-15 3:36 ` Lars Ingebrigtsen
2019-08-15 4:44 ` Drew Adams [this message]
2019-08-15 14:28 ` Eli Zaretskii
2019-08-15 18:47 ` Juri Linkov
2019-08-15 23:37 ` Lars Ingebrigtsen
2019-08-16 0:06 ` Drew Adams
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=d87f3b7b-c067-498e-bda8-a05565780f69@default \
--to=drew.adams@oracle.com \
--cc=15427@debbugs.gnu.org \
--cc=larsi@gnus.org \
/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.