unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Jim Porter <jporterbugs@gmail.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: luangruo@yahoo.com, emacs-devel@gnu.org
Subject: Re: Making `x-underline-at-descent-line' a face attribute
Date: Thu, 6 Jan 2022 09:47:33 -0800	[thread overview]
Message-ID: <d0f4c51e-2448-0439-1ce1-1311c9e417f4@gmail.com> (raw)
In-Reply-To: <83sfu18dao.fsf@gnu.org>

On 1/6/2022 12:43 AM, Eli Zaretskii wrote:
>> From: Jim Porter <jporterbugs@gmail.com>
>> Date: Wed, 5 Jan 2022 21:08:57 -0800
>>
>> What about enhancing the :box attribute further to allow different
>> widths (and possibly colors) for each of the 4 sides?
> 
> The width part is already supported.  We could make the value of
> :color be a cons cell as well, if that is useful (is it?)

Looking at the docs for :box, I only see the ability to set separate 
vertical and horizontal widths (but not to set all 4 sides to different 
values). The latter would be useful in some cases, for example to make 
the tab-bar tabs look more like tabs. You could specify a box with a 
border on all sides but the bottom for the active tab, and a border 
*only* on the bottom for inactive tabs, so you get something like:

               +----------+
     inactive  |  active  |  inactive
   ------------+          +------------

One benefit of making :color a cons cell is that it would be possible to 
create an effect like `:style released-button', but allow the programmer 
to control the exact colors to use. That could be useful in some 
situations. For instance, the modeline could have a border at the top 
and bottom (but not the sides) with different colors to make it appear 
raised. I tried using `:style released-button' for that, but it appears 
to only support a :box with non-zero widths on all sides.

> But mixing :box and :underline is IMO not a good idea.  They are
> separate attributes.

I think I agree. From the original message, it sounded like the goal was 
to get a border around the modeline, so using :underline isn't the ideal 
option semantically. If :box were more flexible, it'd be possible to use 
that to draw borders around things, even if that "border" was just on 
the bottom (making it look similar to an underline).



  reply	other threads:[~2022-01-06 17:47 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <87lezt5v7h.fsf.ref@yahoo.com>
2022-01-06  4:44 ` Making `x-underline-at-descent-line' a face attribute Po Lu
2022-01-06  5:08   ` Jim Porter
2022-01-06  5:35     ` Po Lu
2022-01-06  8:47       ` Eli Zaretskii
2022-01-06  8:43     ` Eli Zaretskii
2022-01-06 17:47       ` Jim Porter [this message]
2022-01-06 20:07         ` Eli Zaretskii
2022-01-06 20:18           ` Juri Linkov
2022-01-07  0:55             ` Po Lu
2022-01-07  7:17               ` Eli Zaretskii
2022-01-07  8:40               ` Juri Linkov
2022-01-07  9:40                 ` Stefan Kangas
2022-01-08 18:20                   ` Juri Linkov
2022-01-08 18:52                     ` Eli Zaretskii
2022-01-08 19:09                       ` Juri Linkov
2022-01-08 19:14                         ` Eli Zaretskii
2022-01-07 13:14                 ` Eli Zaretskii
2022-01-07 13:32                   ` martin rudalics
2022-01-08 18:23                   ` Juri Linkov
2022-01-08 18:39                     ` Eli Zaretskii
2022-01-06 20:49           ` Jim Porter
2022-01-07  6:43             ` Eli Zaretskii
2022-01-06  8:44   ` Eli Zaretskii
2022-01-06  9:36     ` Po Lu
2022-01-06 12:02       ` Eli Zaretskii
2022-01-06 12:24         ` Po Lu
2022-01-06 13:51           ` Eli Zaretskii
2022-01-07  0:47             ` Po Lu
2022-01-07  7:12               ` Eli Zaretskii
2022-01-07  7:14                 ` Po Lu
2022-01-07  8:28                   ` Eli Zaretskii
2022-01-07 11:04                     ` Po Lu
2022-01-07 12:21                       ` Eli Zaretskii
2022-01-07 12:33                         ` Po Lu
2022-01-07 12:41                           ` Eli Zaretskii
2022-01-09  0:43                       ` Po Lu
2022-01-10 10:54                         ` Po Lu
2022-01-10 11:32                           ` Po Lu
2022-01-10 17:41                           ` Eli Zaretskii
2022-01-11  0:47                             ` Po Lu
2022-01-11 16:46                               ` Eli Zaretskii
2022-01-06  8:48   ` Lars Ingebrigtsen
2022-01-06  9:38     ` Po Lu

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=d0f4c51e-2448-0439-1ce1-1311c9e417f4@gmail.com \
    --to=jporterbugs@gmail.com \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=luangruo@yahoo.com \
    /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).