GNU bug report logs -
#61413
[PATCH] Make warnings show a "warning" emoji instead of a stop-sign
Previous Next
Full log
Message #129 received at 61413 <at> debbugs.gnu.org (full text, mbox):
On Mon, 2025-03-10 at 18:53 +0200, Eli Zaretskii wrote:
> > From: Konstantin Kharlamov <Hi-Angel <at> yandex.ru>
> > Cc: rudolf <at> adamkovic.org, rms <at> gnu.org, maurooaranda <at> gmail.com,
> > stefankangas <at> gmail.com, 61413 <at> debbugs.gnu.org
> > Date: Mon, 10 Mar 2025 12:02:01 +0300
> >
> > On Sat, 2025-03-08 at 18:05 +0100, Kévin Le Gouguec wrote:
> > > Eli Zaretskii <eliz <at> gnu.org> writes:
> > >
> > > > > > If you want me to install some patches, please post them.
> > > > >
> > > > > Thanks for the nudge; re-attaching the patch that makes
> > > > > button.el
> > > > > preserve help-echo strings; reviewed by Mauro in two parts¹²,
> > > > > then
> > > > > squashed³. I can install it if we think that fix is
> > > > > unambiguously good.
> > > >
> > > > LGTM, thanks.
> > >
> > > Just pushed, thanks for the review.
> > >
> > > I'll defer to other participants regarding what should be done
> > > next -
> > > e.g. (a) should ⛔ be given a second chance now that it has a
> > > help-
> > > echo
> > > string, making its function more discoverable; (b) should we use
> > > another
> > > symbol; (c) should we eschew images entirely for that function &
> > > use
> > > a
> > > text button.
> >
> > Sorry for stupid question, but… I just wanted to see how do
> > warnings
> > look now, and I figured I don't know how to produce the buffer
> > being
> > discussed 😅
>
> Like this:
>
> M-: (display-warning 'warning "foo") RTE
Thanks! So, I'm looking at the buffer, and I don't see much difference
compared to how it was before, so my vote in preference of either
applying the patch (that one that started the discussion) or making the
icon a button with text instead still stands.
This bug report was last modified 56 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.