embark icon indicating copy to clipboard operation
embark copied to clipboard

Add embark-context-menu

Open minad opened this issue 6 months ago • 4 comments

(context-menu-mode 1)
(add-hook 'context-menu-functions #'embark-context-menu 100)

minad avatar Jun 03 '25 00:06 minad

See also:

  • https://github.com/minad/marginalia/commit/eb46a165758aa53727a2dbf2ad04cb0750385b49
  • https://github.com/minad/vertico/commit/c82f09f9dc1d90aa22728792836551aa4827fb60

minad avatar Jun 03 '25 11:06 minad

@oantolin What do you think about this change? It is a relatively unobtrusive addition, and purely optional. I started using the context menu a bit. I like it for discoverability, and also for onboarding new users.

minad avatar Jun 05 '25 10:06 minad

It doesn't get in the way of people who don't use the context menu, so I'm all for it. Let me test it a bit and i'll get back to you.

oantolin avatar Jun 05 '25 18:06 oantolin

Maybe you have some improvement ideas too - about how the target is presented. Right now the menu entries are fixed, but the target can be seen as tooltip. But tbh I find tooltips on menus sometimes a little bit too much.

minad avatar Jun 05 '25 19:06 minad

Friendly ping @oantolin :)

minad avatar Jun 19 '25 10:06 minad

I've finally gotten around to testing this, and I like it. Putting the actions in the menu always felt very overwhelming to me. This is a nice compromise. And I'd leave the tooltips as-is, they do add to the experience and are not too long.

oantolin avatar Jun 22 '25 05:06 oantolin