[PATCH] Improve default mouse menu for mode from modeline

[PATCH] Improve default mouse menu for mode from modeline

[Pierre Téchoueyres]

[-- Attachment #1: Type: text/plain, Size: 836 bytes --]

Hello emacs's developpers,

I would submit an enhacement for when you use the mouse to display the
menu for a mode (major or minor) from the modeline.


For now, if the mode doesn't define a custom mouse menu, the menu title
simply display the mode indicator. But sometimes this indicator isn't
really obvious.

The attached patch try to improve this by appending the full mode name
after the indicator.

For example the Smartparens minor mode has 'SP' as indicator. So his
menu is look like:
---------------------
| SP
---------------------
| Turn off minor mode
| Help for minor mode
----------------------

Whit the attached patch the menu will look like:

------------------------
| SP - Smartparens Mode
------------------------
| Turn off minor mode
| Help for minor mode
------------------------

Hope this could help.

Pierre.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-minor-mode-menu-from-indicator-now-display-full-mino.patch --]
[-- Type: text/x-patch, Size: 2083 bytes --]

From 8ccd620d2891eee0ec1910a002adca0ab657f7ab Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pierre=20T=C3=A9choueyres?= <pierre.techoueyres@free.fr>
Date: Wed, 10 Oct 2018 20:08:05 +0200
Subject: [PATCH] 'minor-mode-menu-from-indicator' now display full minor mode.

When there is no menu for a mode, display the mode name after the
indicator instead of just the indicator (which is sometime cryptic).
Ex:
before : SP
now    : SP - Smartparens Mode

* etc/NEWS: Add en entry for this new feature.
* lisp/mouse.el (minor-mode-menu-from-indicator): Append the mode name
  after the indicator when there is no menu defined by the mode.
---
 etc/NEWS      | 9 ++++++++-
 lisp/mouse.el | 5 ++++-
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/etc/NEWS b/etc/NEWS
index ee74e86f40..739674cf3e 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -824,11 +824,18 @@ UUID at point.
 
 
 ** Interactive automatic highlighting
-
 +++
 *** 'highlight-regexp' can now highlight subexpressions.
 The now command accepts a prefix numeric argument to choose the
 subexpression.
+** Mouse display major mode menu
++++
+*** 'minor-mode-menu-from-indicator' now display full minor mode.
+When there is no menu for a mode, display the mode name after the
+indicator instead of just the indicator (which is sometime cryptic).
+Ex:
+before : SP
+now    : SP - Smartparens Mode
 
 \f
 * New Modes and Packages in Emacs 27.1
diff --git a/lisp/mouse.el b/lisp/mouse.el
index cb63ca51c5..2aba9bf458 100644
--- a/lisp/mouse.el
+++ b/lisp/mouse.el
@@ -169,7 +169,10 @@ minor-mode-menu-from-indicator
                 (mouse-menu-non-singleton menu)
               (if (fboundp mm-fun)      ; bug#20201
                   `(keymap
-                    ,indicator
+                    ,(format "%s - %s" indicator
+			     (capitalize
+			      (replace-regexp-in-string
+			       "-" " " (format "%S" minor-mode))))
                     (turn-off menu-item "Turn off minor mode" ,mm-fun)
                     (help menu-item "Help for minor mode"
                           (lambda () (interactive)
-- 
2.17.1

Re: [PATCH] Improve default mouse menu for mode from modeline

[Pierre Téchoueyres]

Hello,
My proposal doesn't seem to have received any interest.
Should I submit a bug (as an enhancement request) report for this
instead ?

pierre.techoueyres@free.fr (Pierre Téchoueyres) writes:

> Hello emacs's developpers,
>
> I would submit an enhacement for when you use the mouse to display the
> menu for a mode (major or minor) from the modeline.
>
>
> For now, if the mode doesn't define a custom mouse menu, the menu title
> simply display the mode indicator. But sometimes this indicator isn't
> really obvious.
>
> The attached patch try to improve this by appending the full mode name
> after the indicator.
>
> For example the Smartparens minor mode has 'SP' as indicator. So his
> menu is look like:
> ---------------------
> | SP
> ---------------------
> | Turn off minor mode
> | Help for minor mode
> ----------------------
>
> Whit the attached patch the menu will look like:
>
> ------------------------
> | SP - Smartparens Mode
> ------------------------
> | Turn off minor mode
> | Help for minor mode
> ------------------------
>
> Hope this could help.
>
> Pierre.

Re: [PATCH] Improve default mouse menu for mode from modeline

[Eli Zaretskii]

> From: pierre.techoueyres@free.fr (Pierre Téchoueyres)
> Date: Sat, 27 Oct 2018 12:47:14 +0200
> 
> My proposal doesn't seem to have received any interest.
> Should I submit a bug (as an enhancement request) report for this
> instead ?

Yes, submitting an enhancement request is always better.

Thanks.

Re: [PATCH] Improve default mouse menu for mode from modeline

[Stefan Monnier]

> My proposal doesn't seem to have received any interest.

FWIW, I think it's a good idea.


        Stefan

Re: [PATCH] Improve default mouse menu for mode from modeline

[Pierre Téchoueyres]

Hello Stefan,
Thanks for your support. I opened the bug #33188. Hope it will be
merged in some days.


Stefan Monnier <monnier@iro.umontreal.ca> writes:

>> My proposal doesn't seem to have received any interest.
>
> FWIW, I think it's a good idea.
>
>
>         Stefan