[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#69237: 30.0.50; Toggle password visibility
From: |
Michael Albinus |
Subject: |
bug#69237: 30.0.50; Toggle password visibility |
Date: |
Fri, 23 Feb 2024 16:43:17 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
Kévin Le Gouguec <kevin.legouguec@gmail.com> writes:
Hi Kévin,
> No strong feelings re. keybindings; I've been using C-c C-c because I
> wanted something easy to mash, but TAB fits that criterion as well. Not
> sure we can leverage any well-established mnemonic from other keymaps;
> FWIW…
>
> * TAB could be familiar to some outline users, but they'll probably
> associate that binding to (un)folding, not necessarily hiding/revealing;
>
> * hideshow has multiple bindings for "toggle-hiding", so not sure we can
> count on any single one having enough "mindshare".
>
> (Not too worried about needing to input TAB verbatim - I tend to
> naturally reach for C-q TAB when I need to do that, since in most
> contexts Emacs has a useful command bound to TAB instead of
> self-insert-command)
There are not so many comments on the keybindings. I guess this is
because only few people read the bug-gnu-emacs ML. Let's install this on
master, and see who hollers :-)
> Other than the keybinding, the only other musing I have is regarding the
> indicator: wondering if icons.el could help trim some of the boilerplate
> down. In any case, it would give the user some degree of control over
> how the indicator is shown (via the icon-preference option) even if we
> don't provide 'emoji nor 'symbol variants.
>
> Not overly familiar with that library though - if that's something you
> think might be worth looking into, the define-icon forms in outline.el
> might help you assess how helpful the library would actually be.
That's a good idea. I've implemented it, see the reworked patch
appended. And now with documentation.
Best regards, Michael.
txtVupPtVN5Ol.txt
Description: Text Data
- bug#69237: 30.0.50; Toggle password visibility, Michael Albinus, 2024/02/18
- bug#69237: 30.0.50; Toggle password visibility, Kévin Le Gouguec, 2024/02/22
- bug#69237: 30.0.50; Toggle password visibility,
Michael Albinus <=
- bug#69237: 30.0.50; Toggle password visibility, Kévin Le Gouguec, 2024/02/23
- bug#69237: 30.0.50; Toggle password visibility, Michael Albinus, 2024/02/25
- bug#69237: 30.0.50; Toggle password visibility, Kévin Le Gouguec, 2024/02/25
- bug#69237: 30.0.50; Toggle password visibility, Michael Albinus, 2024/02/25
- bug#69237: 30.0.50; Toggle password visibility, Kévin Le Gouguec, 2024/02/25
- bug#69237: 30.0.50; Toggle password visibility, Eli Zaretskii, 2024/02/25
- bug#69237: 30.0.50; Toggle password visibility, Michael Albinus, 2024/02/25
- bug#69237: 30.0.50; Toggle password visibility, Eli Zaretskii, 2024/02/25
- bug#69237: 30.0.50; Toggle password visibility, Michael Albinus, 2024/02/25
- bug#69237: 30.0.50; Toggle password visibility, Kévin Le Gouguec, 2024/02/25