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

Previous Next

Full log

View this message in rfc822 format

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

>> 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'.

Previous Next

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