From: Dmitry Gutov <dgutov@yandex.ru>
To: Juri Linkov <juri@linkov.net>
Cc: 47012@debbugs.gnu.org
Subject: bug#47012: xref copies keymap properties to minibuffer
Date: Thu, 1 Apr 2021 17:13:52 +0300 [thread overview]
Message-ID: <4d3edf81-97d8-4e0e-51c0-7b2d71b57889@yandex.ru> (raw)
In-Reply-To: <87zgyiv03f.fsf@mail.linkov.net>
On 01.04.2021 11:43, Juri Linkov wrote:
>> https://altbox.dev/ack/screenshot.png
>> https://spinorlab.wordpress.com/2015/08/15/the-silver-searcher/
>
> But all of them display green foreground file names
> with yellow background matches. This is exactly
> what grep.el does by default, so it's the reason
> why output of grep.el is readable, but output of xref.el is not.
3/4 use something yellow-ish, yes (ripgrep in the console uses red
foreground, for some reason).
> The problem is that currently in xref both file names and matches
> use the same color green. The only difference is that file names
> are displayed with green foreground, and matches with green background.
Okay. Guess I haven't experienced that because in my theme highlight
looks grey-ish. *sigh*
> What causes the difficulty in readability is the effect similar to
> https://en.wikipedia.org/wiki/Stroop_effect that causes the
> delay in reaction time between congruent and incongruent stimuli.
>
> Using exactly the same grep colors in xref by changing
> 'xref-match' to inherit from the 'match' face
> completely solves this problem.
You are right. I could even say "unfortunately", because IMHO the bright
yellow highlights are too much. Too strong emphasis, visually.
So let's change it to inherit from 'match', because that's what that
face is documented to be used for.
Additionally, what do you think about toning down 'match''s background
color? Maybe use some subtler yellow like "lemon chiffon" or "khaki1"?
Or "light goldenrod".
ag and deadgrep use some darker khaki tones on the screenshots, but both
probably assume dark background.
>>> Another suggestion how to remove "visual garbage" is to truncate
>>> duplicate prefixes: currently the prefixes of long absolute file names
>>> are repeated for all file names. It would improve readability
>>> to display shorter relative file names without duplicate project root part.
>>
>> Please try (setq xref-file-name-display 'project-relative).
>
> Thanks, I didn't know about this. Shouldn't this be the default value
> since this is what's displayed by grep and ripgrep.
I wouldn't mind, personally.
> Actually,
> there is no exact option for what grep and ripgrep do,
> because they display file names relative to the search directory.
> But currently there is no xref option to display file names
> relative to the subdirectory specified by 'C-u C-x p g'.
This issue is tricky because xref-find-definitions does not assume the
presence of a project, or even of any kind of containing directory. And
yet, it's handy to show its results with relative file names when
possible, too. So I picked "relative to the project" as the option
value, and the corresponding logic.
I think what you're talking about is only a problem when the directory
has no containing project at all. In that case we could probably default
to the value of default-directory as the reference.
Not sure the logic is applicable to general xref output, though (e.g.
xref-find-definitions search in a directory with TAGS but without .git).
So perhaps that behavior should be opt-in, e.g. with an extra arg for
xref--show-xrefs.
Or just keep it an exception, hoping people don't use 'C-u C-x p g'
outside of projects often enough for this to be a nuisance.
next prev parent reply other threads:[~2021-04-01 14:13 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-08 20:03 bug#47012: xref copies keymap properties to minibuffer Juri Linkov
2021-03-09 2:08 ` Dmitry Gutov
2021-03-11 20:58 ` Juri Linkov
2021-03-24 20:38 ` Juri Linkov
2021-03-24 21:57 ` Dmitry Gutov
2021-03-25 9:40 ` Juri Linkov
2021-03-25 10:57 ` Dmitry Gutov
2021-03-25 21:28 ` Juri Linkov
2021-03-25 22:12 ` Dmitry Gutov
2021-03-30 19:16 ` Juri Linkov
2021-03-31 15:47 ` Dmitry Gutov
2021-03-31 15:59 ` Eli Zaretskii
2021-03-31 16:10 ` Dmitry Gutov
2021-03-31 16:57 ` Eli Zaretskii
2021-04-01 0:25 ` Dmitry Gutov
2021-04-01 7:17 ` Eli Zaretskii
2021-03-31 17:05 ` Juri Linkov
2021-04-01 1:16 ` Dmitry Gutov
2021-04-01 8:43 ` Juri Linkov
2021-04-01 14:13 ` Dmitry Gutov [this message]
2021-04-01 18:45 ` Juri Linkov
2021-04-01 19:06 ` Eli Zaretskii
2021-04-01 21:28 ` Dmitry Gutov
2021-04-02 6:08 ` Eli Zaretskii
2021-04-02 23:50 ` Dmitry Gutov
2021-04-03 7:24 ` Eli Zaretskii
2021-04-03 18:12 ` Dmitry Gutov
2021-04-03 18:19 ` Eli Zaretskii
2021-04-02 8:20 ` Juri Linkov
2021-04-01 22:43 ` Dmitry Gutov
2021-04-02 8:25 ` Juri Linkov
2021-04-02 23:23 ` Dmitry Gutov
2021-04-04 22:55 ` Juri Linkov
2021-04-05 2:15 ` Dmitry Gutov
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=4d3edf81-97d8-4e0e-51c0-7b2d71b57889@yandex.ru \
--to=dgutov@yandex.ru \
--cc=47012@debbugs.gnu.org \
--cc=juri@linkov.net \
/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).