unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Filipe Silva <filipe.silva@gmail.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: James Nguyen <james@jojojames.com>,
	Emacs developers <emacs-devel@gnu.org>
Subject: Re: Native line numbers, final testing
Date: Tue, 4 Jul 2017 11:50:23 -0300	[thread overview]
Message-ID: <CAEwkUWNPNBQC-t8FBGBU-E3=1n6gJcEy=OxSwvU2GHrUuL4W5Q@mail.gmail.com> (raw)
In-Reply-To: <CAEwkUWMONz12guQxP+=uHb9ASpWU2R=Hg=x4UyuNnGpM21pBLg@mail.gmail.com>

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

I don't know why the message was garbled, before. here's another try:

Eli thanks you for the information. will test that.

I found a bug regarding the line-number-current-line face. Here's a
screenshot: http://i.imgur.com/I2sneeL.png


I'm using this configuration for relative visual numbers:

(custom-set-variables
...
 '(display-line-numbers 'visual)
...)

And here's my setup for the faces:

  ;; MODE SUPPORT: line numbers and relative line numbers

  (line-number              (:inherit 'highlight-numbers-number))
  (line-number-current-line (:foreground gruvbox-neutral_red))

In my testing, I found that when point is on line 1, everything is fine. As
you go down line by line, the line-number-current-face skips ahead by one
so by the time point is on line 10, line-number-current-line face is really
acting on a number 9 lines below what should be the current line.


On Tue, Jul 4, 2017 at 11:48 AM, Filipe Silva <filipe.silva@gmail.com>
wrote:

> Eli thanks you for the information. will test that.
>
> I found a bug regarding the line-number-current-line face. Here's a
> screenshot: http://i.imgur.com/I2sneeL.png
>
> I'm using this configuration for relative visual numbers:
>
> (custom-set-variables
> ...
>  '(display-line-numbers 'visual)
> ...)
>
> And here's my setup for the faces:
>
>   ;; MODE SUPPORT: line numbers and relative line numbers
>
>   (line-number              (:inherit 'highlight-numbers-number))
>   (line-number-current-line (:foreground gruvbox-neutral_red))
>
> In my testing, I found that when point is on line 1, everything is fine.
> As you go down line by line, the line-number-current-face skips ahead by
> one so by the time point is on line 10, line-number-current-line face is
> really acting on a number 9 lines below what should be the current line.
>
>
> On Tue, Jul 4, 2017 at 11:32 AM, Eli Zaretskii <eliz@gnu.org> wrote:
>
>> > From: Filipe Silva <filipe.silva@gmail.com>
>> > Date: Tue, 4 Jul 2017 07:57:10 -0300
>> > Cc: Eli Zaretskii <eliz@gnu.org>, Emacs developers <emacs-devel@gnu.org
>> >
>> >
>> > When working with really large files, 10k+ lines long, having the
>> current line display the abs line number can
>> > feel "to much" because all lines have max 2 number and the current line
>> has 5 number so the fringe has to
>> > grow extra 3 columns just to acommodate the current abs line number of
>> the current line.
>> >
>> > Would it be possible to have like a toggle mechanism to switch the
>> current line number of the current line from
>> > abs to nothing or even to zero? that'd be a great addition to this
>> already awesome feature.
>>
>> You can customize display-line-numbers-current-absolute to do what you
>> want.
>>
>> Thank you for the feedback.
>>
>
>

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

  reply	other threads:[~2017-07-04 14:50 UTC|newest]

Thread overview: 90+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-30 14:49 Native line numbers, final testing Eli Zaretskii
2017-06-30 17:51 ` Alex
2017-06-30 18:20   ` Eli Zaretskii
2017-06-30 19:06     ` Alex
2017-06-30 19:55       ` Eli Zaretskii
2017-06-30 21:15         ` Alex
2017-07-01  8:00           ` Eli Zaretskii
2017-07-01 21:00             ` Alex
2017-07-02  2:40               ` Eli Zaretskii
2017-07-02  5:16                 ` Alex
2017-07-02 15:10                   ` Eli Zaretskii
2017-07-02 16:47                     ` Stefan Monnier
2017-07-02 16:51                       ` Eli Zaretskii
2017-07-02 17:38                         ` Stefan Monnier
2017-07-02 19:27                           ` Eli Zaretskii
2017-07-03  5:06                     ` Alex
2017-07-03 15:24                       ` Eli Zaretskii
2017-07-04 19:36                         ` Alex
2017-07-05 17:39                           ` Eli Zaretskii
2017-07-07  2:46                             ` Alex
2017-07-07  6:19                               ` Eli Zaretskii
2017-07-07  9:24                                 ` Eli Zaretskii
2017-07-08 20:51                                   ` Alex
2017-07-09 20:16                                     ` James Cloos
2017-07-09 21:45                                       ` Yuri Khan
2017-07-10  2:33                                         ` Eli Zaretskii
2017-07-10  7:09                                           ` Yuri Khan
2017-07-10 17:02                                             ` Eli Zaretskii
2017-07-10  2:31                                       ` Eli Zaretskii
2017-07-10  5:35                                         ` James Cloos
2017-07-10 17:00                                           ` Eli Zaretskii
2017-07-10 18:15                                             ` Filipe Silva
2017-07-10 18:18                                               ` Eli Zaretskii
2017-07-10 18:23                                                 ` Filipe Silva
2017-07-10 18:32                                             ` James Cloos
2017-07-11 20:58                                             ` Alex
2017-07-11 21:18                                               ` Filipe Silva
2017-07-11 21:20                                                 ` Filipe Silva
2017-07-11 21:37                                                   ` Alex
2017-07-12  2:35                                               ` Eli Zaretskii
2017-07-12  2:53                                                 ` Alex
2017-07-12 14:21                                                   ` Eli Zaretskii
2017-07-12 17:22                                                     ` Alex
2017-07-12 17:25                                                       ` Alex
2017-07-12 18:38                                                       ` Eli Zaretskii
2017-07-12 20:03                                                         ` Alex
2017-07-13  2:38                                                           ` Eli Zaretskii
2017-07-13  4:11                                                             ` Alex
2017-07-13 15:56                                                               ` Eli Zaretskii
2017-07-26  3:50                                                                 ` Alex
2017-07-26 14:42                                                                   ` Eli Zaretskii
2017-07-29  6:12                                                                     ` Alex
2017-07-29  7:01                                                                       ` Eli Zaretskii
2017-07-07  9:47                                 ` Eli Zaretskii
2017-07-07  9:49                                   ` Eli Zaretskii
2017-07-07 11:14                                     ` Filipe Silva
2017-07-07 12:21                                       ` Eli Zaretskii
2017-07-07 12:29                                   ` Eli Zaretskii
     [not found]                                     ` <CAEwkUWN8GkCyOiF4jEgKuZwJHhvMgJi9yVnvggRvu+Yddfp4qQ@mail.gmail.com>
2017-07-07 12:56                                       ` Filipe Silva
2017-07-01  1:59 ` Filipe Silva
2017-07-02 19:27 ` James Nguyen
2017-07-03  2:33   ` Eli Zaretskii
2017-07-03  3:22     ` James Nguyen
2017-07-03 15:58       ` Eli Zaretskii
2017-07-03 17:04         ` James Nguyen
2017-07-04 10:57           ` Filipe Silva
2017-07-04 11:00             ` Filipe Silva
2017-07-04 13:51               ` Kaushal Modi
2017-07-04 14:30               ` Eli Zaretskii
2017-07-04 14:32             ` Eli Zaretskii
2017-07-04 14:48               ` Filipe Silva
2017-07-04 14:50                 ` Filipe Silva [this message]
2017-07-04 15:44                   ` Eli Zaretskii
2017-07-04 16:22                     ` Filipe Silva
2017-07-04 16:34                       ` Filipe Silva
2017-07-04 16:35                       ` Richard Copley
2017-07-04 16:44                         ` Eli Zaretskii
2017-07-04 17:13                           ` Richard Copley
2017-07-04 17:35                             ` Filipe Silva
2017-07-04 17:48                               ` Eli Zaretskii
2017-07-04 17:52                               ` Stefan Monnier
2017-07-10 18:22                                 ` Filipe Silva
2017-07-10 20:28                                   ` Stefan Monnier
2017-07-04 17:47                             ` Eli Zaretskii
2017-07-04 17:50                           ` Alex
2017-07-04 18:24                             ` Eli Zaretskii
2017-07-04 18:37                               ` Richard Copley
2017-07-04 18:43                                 ` Eli Zaretskii
2017-07-05 20:24 ` Andy Moreton
2017-07-06 17:24   ` Eli Zaretskii

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='CAEwkUWNPNBQC-t8FBGBU-E3=1n6gJcEy=OxSwvU2GHrUuL4W5Q@mail.gmail.com' \
    --to=filipe.silva@gmail.com \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=james@jojojames.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).