unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: "Jarosław Rzeszótko" <sztywny@gmail.com>
To: Drew Adams <drew.adams@oracle.com>
Cc: 16192@debbugs.gnu.org
Subject: bug#16192: hl-line-mode: Current line highlighting takes precedence over region highlighting
Date: Thu, 19 Dec 2013 18:41:51 +0100	[thread overview]
Message-ID: <CAHnYAZaNNAerXnNk_7_gTre61ejzjFTPUcKWxjMadx4ZSigr2w@mail.gmail.com> (raw)
In-Reply-To: <4497f846-4399-48b4-bd48-4bf15a44be81@default>

[-- Attachment #1: Type: text/plain, Size: 1145 bytes --]

Correction from my side, the last problem seems to actually be independent
of the fix, I reproduced it with vanilla emacs from vcs, no hl-line-mode,
emacs ran with --no-init, without any .Xresources etc., just:

(setq-default left-margin-width 1)

open a new buffer, select a few lines, and you get this:

http://imgur.com/M1LFjeF


2013/12/19 Drew Adams <drew.adams@oracle.com>

> > I've just installed a patch which lowers the priority of the hl-line
> > highlighting, which seems like a good thing to do in general and which
> > should resolve this problem (unless you use the same attributes for the
> > region and the hl-line faces).
>
> It's not about the priority of the hl-line highlighting. (And other code
> might well expect the priority that was there before.)
>
> It's about the priority of the region highlighting.  Raise that,
> instead of lowering other priorities case by case (and possibly breaking
> other code).  It is the region overlay that is new.  It is its priority
> that needs fixing.
>
> But didn't you say that you didn't want to be fiddling with priorities,
> to deal with the problems introduced by this change?
>

[-- Attachment #2: Type: text/html, Size: 1644 bytes --]

  reply	other threads:[~2013-12-19 17:41 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-19 11:38 bug#16192: hl-line-mode: Current line highlighting takes precedence over region highlighting Jarosław Rzeszótko
2013-12-19 14:02 ` Stefan Monnier
2013-12-19 14:19   ` Jarosław Rzeszótko
2013-12-19 16:43   ` Drew Adams
2013-12-19 17:41     ` Jarosław Rzeszótko [this message]
2013-12-19 19:27       ` Eli Zaretskii
2013-12-19 19:47         ` Jarosław Rzeszótko
2013-12-19 20:31           ` Stefan Monnier
2013-12-19 18:02     ` Stefan Monnier
     [not found]       ` <bcaf51ae-dec7-4ac3-87f8-0142ababbc9c@default>
2014-06-17 14:30         ` Drew Adams
2014-06-17 15:07         ` Stefan Monnier
2013-12-19 16:38 ` 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

  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=CAHnYAZaNNAerXnNk_7_gTre61ejzjFTPUcKWxjMadx4ZSigr2w@mail.gmail.com \
    --to=sztywny@gmail.com \
    --cc=16192@debbugs.gnu.org \
    --cc=drew.adams@oracle.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).