GNU bug report logs - #77224
[PATCH] New minor mode 'cursor-indicators-mode'

Previous Next

Full log

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

From: Daniel Colascione <dancol <at> dancol.org>
To: Juri Linkov <juri <at> linkov.net>
Cc: 77224 <at> debbugs.gnu.org,
 Elijah Gabe Pérez <eg642616 <at> gmail.com>,
 Eli Zaretskii <eliz <at> gnu.org>, JD Smith <jdtsmith <at> gmail.com>,
 mail <at> daniel-mendler.de
Subject: Re: bug#77224: [PATCH] New minor mode 'cursor-indicators-mode'
Date: Sun, 30 Mar 2025 14:04:40 -0400

Juri Linkov <juri <at> linkov.net> writes:

>>> Please note that this is insufficient for 'repeat-mode'.
>>> Currently it requires adding an advice to update the cursor
>>> on the repeat timer events:
>>
>> I solved this problem by wrapping the `repeat-echo-function' and `after-focus-change-function'.
>>
>> https://gist.github.com/jdtsmith/a169362879388bc1bdf2bbb977782d4f
>
> Interesting to see that you switched from `post-command-hook'
> to `repeat-echo-function' and `after-focus-change-function'.
>
> So we could have an option that allows switching between 1) idle timers,
> 2) 'post-command-hook', 3) a list of more specific hooks.
>
> In any case we still need to add such a condition:
>
>   (if (memq 'repeat-in-progress cursor-indicators-conditions)
>       (add-function :after repeat-echo-function #'cursor-indicators--update-indications)
>    (remove-function :after repeat-echo-function #'cursor-indicators--update-indications))
>
> that have to be added only when the customized value contains 'repeat-in-progress'.

This is exactly the sort of complexity that nobody but those on this
thread will care about and that we need to abstract behind some
stupid-easy DWIM interface.

Previous Next

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