[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#61413: [PATCH] Make warnings show a "warning" emoji instead of a sto
From: |
Kévin Le Gouguec |
Subject: |
bug#61413: [PATCH] Make warnings show a "warning" emoji instead of a stop-sign |
Date: |
Sat, 15 Feb 2025 11:26:23 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
Rudolf Adamkovič <rudolf@adamkovic.org> writes:
>> (b) as Konstantin suggested, ditching pictograms entirely, and replacing
>> that icon with a regular text button clearly labeled "suppress"
>
> +1 for a standard, discoverable, and accessible button.
Thoughts prompted by this and rms's messages:
* A bona-fide bug:
display-warning *strips* the button's help-echo property:
(insert (buttonize (icon-string 'warnings-suppress)
#'warnings-suppress type)
'buttonize' clobbers the help-echo property added by 'icon-string' ☹️
That does make the current emoji extra-obscure, since hovering over it
gives no clue as to what it does!
Solutions:
(a) display-warning could retrieve the property from the string returned
by icon-string, then spoon-feeding it back to buttonize,
(b) buttonize could check whether its STRING argument has a help-echo
property before clobbering it with nil,
(c) we invent a sentinel value for buttonize's HELP-ECHO argument to
mean "psst, STRING already has the right help-echo property; use that".
* A general remark:
For users to whom emoji presentation _in general_ is not accessible
enough, the icon-preference option can be set to 'text to always get "a
regular text button clearly labeled <something>".
Thought it bore mention since the present report is about bikeshedding
the graphical representation of the warnings-suppress icon, whereas
Rudolf's reply suggests - to me - a wider wish to be able to opt out of
these representations: this is already solved.
* A caveat for terminals:
As Eli mentions, icons.el already knows to fall back to a suitable
representation on TTYs. In practice though, the Linux TTY can display a
few non-ASCII characters; as a result, here (6.13 if that matters) Emacs
falls back to warnings-suppress's _symbol_ representation…
(symbol ,(if (and (eq system-type 'windows-nt)
(null window-system))
" » "
" ■ ")) ; 👈 that one
… which one could bikeshed too, if one were so inclined 🫣
- bug#61413: [PATCH] Make warnings show a "warning" emoji instead of a stop-sign, Stefan Kangas, 2025/02/11
- bug#61413: [PATCH] Make warnings show a "warning" emoji instead of a stop-sign, Richard Stallman, 2025/02/17
- bug#61413: [PATCH] Make warnings show a "warning" emoji instead of a stop-sign, Rudolf Adamkovič, 2025/02/18
- bug#61413: [PATCH] Make warnings show a "warning" emoji instead of a stop-sign, Stefan Kangas, 2025/02/18
- bug#61413: [PATCH] Make warnings show a "warning" emoji instead of a stop-sign, Konstantin Kharlamov, 2025/02/18
- bug#61413: [PATCH] Make warnings show a "warning" emoji instead of a stop-sign, Stefan Kangas, 2025/02/18