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 Monnier <monnier@iro.umontreal.ca>, Eli Zaretskii <eliz@gnu.org>
Cc: 15899@debbugs.gnu.org
Subject: bug#15899: 24.3.50; regression: `region' overlay is lower priority than default
Date: Sat, 16 Nov 2013 10:01:19 -0800 (PST)	[thread overview]
Message-ID: <67d3dd9d-9abe-488f-80eb-7d647bd81e7e@default> (raw)
In-Reply-To: <jwviovsnt2n.fsf-monnier+emacsbugs@gnu.org>

> > It's enough for that face to specify a background color, no?
> 
> In some cases, yes, because the region's foreground color is
> often unnoticeable (e.g. same as default).

That's only by default.  We should not base too many assumptions
on any particular face attributes for face `region'.  It is user
customizable.  Better to plan for it to be like any other face:
any attributes at all, at least as much as we can.

> >> I most-positive-fixnum-ly hate overlay priorities.
> > No offense, but I think we can live with that downside ;-)
> 
> The downside is not that I hate it, but the reasons why I hate
> it: it's as much a source of problems as a solution.
> `priorities' impose a total ordering, where often there isn't
> one: in some circumstance one overlay should be on top, in
> others it's the other way around.

I don't disagree (and I'm glad that you are giving reasons ;-)).
But what is a better approach?  A total ordering is black & white,
but at least it gives people a degree of control.  And at least
that control is simple: a total ordering is a simple model.

How about giving an example of a problem?  And a solution -
something that solves that problem and gives users more (not
less) control.

> The "default priority" at least is able to handle those things
> sometimes, by making overlays's ordering depending on nesting.

Not sure what that means, and I wish I did understand what you
mean by that.  Can you give a tiny example to illustrate?

> > In any case, the moment you reimplemented the region as an
> > overlay, you got us this issue, because it is inherent in
> > the use of overlays, and cannot be escaped.
> 
> It was present before as well.  The behavior was different
> but was also a source of "priority problems".

Not clear how so.  Can you elaborate?  Are you referring to
the fact that a user who wants to see some other highlighting
(besides isearch) "on top" could not do so?  That I can see.
If you mean something else then I don't know what it is.

> My intuition tells me that if Emacs had use the current system
> for the last 20 years and had just changed to the "region is
> always at the very top", people would complain just as much.

People sometimes complain less when (a) the new behavior is
proposed and explained and (b) they have an opportunity to
question and discuss it.

We are certainly doing that here, now, but this is something
that would be more appropriate for emacs-devel, IMO.  It would
have been better to initiate a discussion and proposal there,
pointing to the bug report and outlining what the behavior
changes would be.  But you've heard this before...





  reply	other threads:[~2013-11-16 18:01 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-14 22:57 bug#15899: 24.3.50; regression: `region' overlay is lower priority than default Drew Adams
2013-11-15  7:41 ` Eli Zaretskii
2013-11-15 13:54   ` Stefan Monnier
2013-11-15 14:40     ` Eli Zaretskii
2013-11-15 15:32       ` Dmitry Gutov
2013-11-15 16:40         ` Eli Zaretskii
2013-11-15 22:35           ` Dmitry Gutov
2013-11-16  8:49             ` Eli Zaretskii
2013-11-16  9:51               ` Jarek Czekalski
2013-11-16 10:42                 ` Eli Zaretskii
2013-11-16 14:43                   ` Jarek Czekalski
     [not found]                 ` <<83ppq0hbln.fsf@gnu.org>
2013-11-16 16:23                   ` Drew Adams
2013-11-16 16:52                     ` Eli Zaretskii
2013-11-16 22:01                     ` Stefan Monnier
2013-11-16 22:42                       ` Drew Adams
2013-11-16 10:25               ` Dmitry Gutov
2013-11-16 11:24                 ` Eli Zaretskii
2013-11-16 13:49                   ` Dmitry Gutov
2013-11-16 16:30                     ` Drew Adams
2013-11-16 16:20                 ` Drew Adams
     [not found]                 ` <<83ob5kh9nb.fsf@gnu.org>
2013-11-16 16:24                   ` Drew Adams
2013-11-16  1:25       ` Stefan Monnier
2013-11-16  9:06         ` Eli Zaretskii
2013-11-16 17:45           ` Stefan Monnier
2013-11-16 18:01             ` Drew Adams [this message]
2013-11-16 22:00               ` Stefan Monnier
2013-11-17 12:25             ` Daniel Colascione
2013-11-17 15:42               ` Stefan Monnier
2014-02-10  4:14         ` Lars Ingebrigtsen
2013-11-15 15:51     ` Drew Adams
2013-11-16  1:26       ` Stefan Monnier
2013-11-16  3:47         ` Drew Adams
2013-11-15 16:53 ` Barry OReilly
     [not found] <<"<20137354-f982-4314-9c09-21a5fbc36557"@default>
     [not found] ` <<"<jwvsiux4vrn.fsf-monnier+emacsbugs"@gnu.org>
     [not found]   ` <<"<87mwl58yvc.fsf"@yandex.ru>
     [not found]     ` <<834n7dipnq.fsf@gnu.org>
     [not found]       ` <<"<83wqk8hgtf.fsf"@gnu.org>
     [not found]         ` <<7031ba1e-2f47-4dd0-908a-938c26016e12@default>
     [not found]           ` <<83k3g8gug7.fsf@gnu.org>
2013-11-16 17:47             ` Drew Adams
     [not found] <<78b8713a-e96f-4b4d-990a-3af59ebdf942@default>
     [not found] ` <<83zjp5h33w.fsf@gnu.org>
2013-11-15 21:21   ` Drew Adams
     [not found] <<00aa91d2-10a2-4a78-bb95-042d1596a41c@default>
     [not found] ` <<8338mxipix.fsf@gnu.org>
2013-11-15 17:14   ` Drew Adams
2013-11-15 19:33     ` Eli Zaretskii
     [not found] <<20137354-f982-4314-9c09-21a5fbc36557@default>
     [not found] ` <<83ob5mi02j.fsf@gnu.org>
     [not found]   ` <<jwvsiux4vrn.fsf-monnier+emacsbugs@gnu.org>
     [not found]     ` <<83bo1liv80.fsf@gnu.org>
2013-11-15 15:55       ` Drew Adams
2013-11-15 16:43         ` Eli Zaretskii
     [not found] <"<20137354-f982-4314-9c09-21a5fbc36557"@default>
     [not found] ` <"<jwvsiux4vrn.fsf-monnier+emacsbugs"@gnu.org>

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=67d3dd9d-9abe-488f-80eb-7d647bd81e7e@default \
    --to=drew.adams@oracle.com \
    --cc=15899@debbugs.gnu.org \
    --cc=eliz@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    /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.