[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#53255: highlight-regexp should show faces with their properties appl
From: |
Kévin Le Gouguec |
Subject: |
bug#53255: highlight-regexp should show faces with their properties applied when selecting a face |
Date: |
Thu, 20 Jan 2022 23:58:43 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/29.0.50 (gnu/linux) |
Lars Ingebrigtsen <larsi@gnus.org> writes:
> So I put the affixion thing into read-file-name and made hi-lock call
> read-file-name instead of doing the completion itself.
Great! A couple of nits:
1. Is there a reason why you went with (list "SAMPLE\t" "" face) instead
of (list face "" \tSAMPLE)? "(elisp) Completion Variables" says that
the completion goes first, then the prefix, then the suffix; I don't
know how hard a rule that is, however I do think the latter option
fares better when faced with variable heights.
Cf. patch and screenshots: I like the second option better because
IIUC the face names (which is what I'll be focusing my eyes on in
order to know what to type; the face decoration I can see well enough
in my peripheral vision) stand a better chance of being aligned.
(Although I don't know if it's the hour getting late or something,
but now that I actually try out C-u M-s h r I just… can't get any
highlighting, with or without my patch on top. The completion works
fine, but when I exit the minibuffer… nothing gets highlighted.
Ugh. Will sleep on it)
2. As you can see on the screenshots, the prompt shows the default face
twice. I think fixing this is as simple as removing format-prompt
(patch also attached), but then again, I'm not thinking straight this
evening it seems.
Thanks for your time.
affixation.patch
Description: Text Data
current.png
Description: PNG image
suggested.png
Description: PNG image
defaults.patch
Description: Text Data
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, ndame, 2022/01/14
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Lars Ingebrigtsen, 2022/01/15
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Kévin Le Gouguec, 2022/01/15
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Lars Ingebrigtsen, 2022/01/15
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Kévin Le Gouguec, 2022/01/15
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Juri Linkov, 2022/01/15
- bug#53255: [External] : bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Drew Adams, 2022/01/15
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Lars Ingebrigtsen, 2022/01/20
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face,
Kévin Le Gouguec <=
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Kévin Le Gouguec, 2022/01/21
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Lars Ingebrigtsen, 2022/01/21
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Kévin Le Gouguec, 2022/01/21
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Lars Ingebrigtsen, 2022/01/22
- bug#53255: [External] : bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Drew Adams, 2022/01/21
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Juri Linkov, 2022/01/22
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Lars Ingebrigtsen, 2022/01/23
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Juri Linkov, 2022/01/23
- bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Lars Ingebrigtsen, 2022/01/24
- bug#53255: [External] : bug#53255: highlight-regexp should show faces with their properties applied when selecting a face, Drew Adams, 2022/01/15