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: Fri, 15 Nov 2013 07:51:38 -0800 (PST)	[thread overview]
Message-ID: <1519bff5-bf9d-42bc-8993-d96153f0004f@default> (raw)
In-Reply-To: <jwvsiux4vrn.fsf-monnier+emacsbugs@gnu.org>

> > In Emacs 24.3, the region highlight was not done by an overlay,
> > but by special code in C.  Now it is an overlay, so the issue
> > of priority creeps in.
> 
> Indeed.  OTOH, I'm not convinced it's a bug, since this "bug" was
> also the fix for another bug.

Which other bug?  Was that bug there ever since the Emacs region
could be highlighted (i.e., several decades old)?  Or was that
bug introduced recently, as the result of some other change?
Maybe there is more than one change that needs to be reverted?

Oh, I see that Dmitry has now answered that question: 
http://debbugs.gnu.org/cgi/bugreport.cgi?bug=15618

Which was NOT a bug, IMO.  It should not have been "fixed".  The
fact that face `region' highlights "on top of" other highlighting
is _on purpose_, i.e., by design.  Countering that is folly, IMO.

What next?  Someone wants to see "theme" highlighting on top of
Isearch overlays?  And someone eagerly jumps in to "fix" that
"bug" too?

> It's a change, there's no doubt about that. Whether it's better
> or worse is not so clear.

Are you saying that you think it might not be a bug (regression)
to use an overlay rather than a text property?  That would be a
reasonable question.

Or are you suggesting that it might not be a bug that selecting
text does not show the selection face throughout the selection?

The latter would be unreasonable.  When you select text you
should always be able to see which text you selected - all of it,
even if it is only one character.  This should be a no-brainer.
The whole point of having a `region' face is to highlight what
is in the region.

(FWIW, I'm not sure I'm in favor of the change to using an
overlay from using a text property either, but that is not what
this bug report is about.  Using the region to select text, and
then replacing face `region' text property with another face or
another text property, or otherwise making use of the property,
can be useful.  Sure, you can make do without `region' as a text
property, but why should you have to?  Why did this behavior
suddenly need to be changed, after four decades or so of use?)

Please:

1. If you keep the use of an overlay, at *least* give users a
choice (e.g. a user option) of whether face `region' is to be
applied as a text property or used for an overlay.

2. When it is used for an overlay, please make its priority
higher than others, with the exception of isearch overlays
(yes, there can sometimes be a use for continuing to show the
region while isearching).  If there are any other exceptions
needed to bring the behavior back to pre-regression, those
would apply also.  (I can't think of any other exceptions,
right now).

These kinds of changes should first be the subject of proposal
and discussion in emacs-devel, and perhaps a user poll.  They
should not be made willy nilly when fixing a bug.  IMHO.





  parent reply	other threads:[~2013-11-15 15:51 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
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 [this message]
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=1519bff5-bf9d-42bc-8993-d96153f0004f@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.