Tor-björn Claesson <tclaes...@gmail.com> writes:
> (defcustom org-cite-basic-follow-actions
> '[["Open"
> ("b" "bibliography entry" org-cite-basic-goto (list (transient-scope)
> 0))]
> ["Copy"
> ("d" "DOI" org-cite-basic-follow.copy-doi)]
> ["Browse"
> ("u" "url" org-cite-basic-follow.browse-url)]]
> "Hepp"
> :group 'org-cite
> :type 'sexp)
Since we can now allow pretty much a custom suffix definition, what
about adding a bit of syntax sugar to the format.
Rather than doing
("b" "bibliography entry" org-cite-basic-goto (list (transient-scope) 0))
we can allow
("b" "bibliography entry" (org-cite-basic-goto !citation !prefix))
where !citation and !prefix can be pattern-matched and automatically
replaced with the appropriate values.
> (transient-define-prefix org-cite-basic-follow (citation &optional prefix)
> [:class transient-columns
> :setup-children org-cite-basic-follow--setup
> :pad-keys t]
> (interactive)
> (if org-cite-basic-follow-ask
> (transient-setup 'org-cite-basic-follow nil nil
> :scope (list citation))
> (org-cite-basic-goto citation prefix)))
>
> Here I removed activating the transient menu if a prefix argument is
> present. If org-cite-basic-follow-ask is nil (the default) and I want
> to call org-cite-basic-goto with a prefix argument, this would cause the
> transient menu to activate, which seems wrong.
I would still prefer some way to force transient version of the command
in some way. I see it very useful for some people to set the proposed
`org-cite-basic-follow-ask' to nil _most of the time_, but occasionally
switch to transient version for non-standard operations.
Maybe we can utilize some other rarely used prefix argument to force
transient menu.
Say, something like C-- C-u <key sequence>. This will make prefix
argument have a value of '(-4).
--
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>
