Den lör 24 jan. 2026 14.13Ihor Radchenko <[email protected]> skrev:
> I think all you need to do is wrapping (command-execute #',command) > into (org-with-point-at ...) and other things that org-attach currently > does. > This works! Thanks, I should have thought of this! (I was secretly hoping that the solution would be to pattern match on the continuation, complete with a link to the manual :-) > > (org-menu-define org-attach () > > "The dispatcher for attachment commands. > > Shows a list of commands defined in `org-attach-commands' using > `org-menu'." > > :menu org-attach--commands-to-transient-specification > > :default-action (command-execute #'org-attach-attach) > > (interactive)) > > Why do you need (interactive)? I think it should be optional. > An earlier version of org-menu-define would add it if it was missing, but I think there was a problem with this? Maybe I misunderstood. > Also, by default, we should show the old menu somehow. > Some users may prefer the old menu for org-attach while using transient > or, generally, org-menu-system, for other menus. > Would it be ok to add a customisation org-attach-use-org-menu, that would default to false? Another version would be to use the original org-attach dispatcher as the default action, but I prefer the first option, this is not the correct function for the default action. What do you think? Cheers, Tor-björn
