GNU bug report logs - #77715
[PATCH] Add ring-bell functions for mode line and header line.

Previous Next

Full log

Message #89 received at 77715 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ship Mints <shipmints <at> gmail.com>
Cc: eg642616 <at> gmail.com, 77715 <at> debbugs.gnu.org, drew.adams <at> oracle.com
Subject: Re: bug#77715: [PATCH] Add ring-bell functions for mode line and
 header line.
Date: Wed, 30 Apr 2025 17:29:14 +0300

> From: Ship Mints <shipmints <at> gmail.com>
> Date: Wed, 30 Apr 2025 10:09:36 -0400
> Cc: eg642616 <at> gmail.com, 77715 <at> debbugs.gnu.org, drew.adams <at> oracle.com
> 
> On Wed, Apr 30, 2025 at 10:06 AM Eli Zaretskii <eliz <at> gnu.org> wrote:
> 
>  >  But this code is not general enough for that.  It was written
>  >  explicitly for additional optional values for ring-bell-functions, as
>  >  the doc strings say.
>  > 
>  > The prototype was derived from private code used in this narrow case but if we're going to adopt the
>  > functionality in core, I think we go the extra mile to make it appropriately general.
> 
>  But then the entire implementation should be revisited and reviewed
>  with that generality in mind.  So please let's talk about that.  Could
>  you or someone else please describe what general features are meant to
>  be implemented based on this functionality?
> 
> I think of face flashing as the face equivalent to 'pulse-momentary-highlight-region'.

Then why don't we use the functions defined in pulse.el in the first
place? why invent a whole new family of functions,l face attributes,
etc.?

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.