all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Dmitry Gutov <dgutov@yandex.ru>
To: 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 15:49:11 +0200	[thread overview]
Message-ID: <528777D7.1060306@yandex.ru> (raw)
In-Reply-To: <83ob5kh9nb.fsf@gnu.org>

On 16.11.2013 13:24, Eli Zaretskii wrote:
> What way is that?  Before region highlighting was reimplemented as an
> overlay, it had a fixed "priority" that couldn't be controlled or
> worked around.  So what exactly did easy-kill expect from that
> behavior?

That behavior was broken on Leo Liu's system (don't know how) and on 
mine too (with some themes only).

See https://github.com/leoliu/easy-kill/issues/3 for more details.

> But the region always has point on one of its ends, so both the places
> where point was and where it is are clearly visible when the region is
> active.

In many times point wasn't at any of region ends. That's how `easy-mark' 
works, it selects some unit of text *around* the point.

> So why is there a need for the easy-mark to be visible in
> that situation (which is transient and therefore short-lived)?

Just for user information. Looks kinda nice.

> The part of the region that overlaps the easy-mark overlay will not be
> visible, if the region's priority is lower.

That just means that easy-mark will need to set its priority higher than 
the *documented* region priority.

> That's not the race I had in mind.  What I had in mind was users
> complaining about their favorite overlay-based features being obscured
> by the region, lobbying the maintainers to increase the priorities of
> those overlays above the region (and possibly also above the easy-mark
> overlay), or increase the priority of the region overlay; then other
> users complaining about the effects of that, and so on and so forth ad
> nauseam.

This will have to be solved on a package-by-package basis, like every 
contentious feature. One user complaint: change the priority of a 
package-specific overlay. More complaints to change it in different 
directions: add a user option that will set the priority that overlay.

 > How can you even assume that the "region overlay priority
> won't change", given the possibility and enough pressure from users?
> Once out of the bottle, this genie cannot be easily put back.

Region priority will have to be set once (to 100, or something), 
documented, and then never changed again. Any changes will have to be 
made to other overlays' priorities.





  reply	other threads:[~2013-11-16 13:49 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 [this message]
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
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=528777D7.1060306@yandex.ru \
    --to=dgutov@yandex.ru \
    --cc=15899@debbugs.gnu.org \
    --cc=eliz@gnu.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.