[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Org-cite: Replace basic follow-processor with transient menu?
From: |
Ihor Radchenko |
Subject: |
Re: Org-cite: Replace basic follow-processor with transient menu? |
Date: |
Sun, 22 Sep 2024 12:50:44 +0000 |
Tor-björn Claesson <tclaesson@gmail.com> writes:
>> I do think that having extended menus for org-open-at-point could be
>> useful. Not by default, but, for example, with a prefix argument.
>>
> This is a good point, but of much larger scope than just replacing the
> follower of the basic citation-processor.
No problem. But my idea may still be used - without prefix argument,
just move to citation record; with prefix argument - invoke the menu.
Follow processors are provided with prefix argument passed to
`org-open-at-point':
(defun org-cite-basic-goto (datum _)...
The "_" argument is the currently ignored prefix argument.
> Maybe it is a good idea to start small, for example provide
> 1. Open bibtex-entry
> 2. Copy DOI
> 3. Opening DOI/ISBN/URL links in browser
>
> Further functionality can easily be added per user, and good solutions
> incorporated by default in the future?
+1.
Ideally, these options should simply be a customization.
> I have played some more with this - would it be a good idea to include
> macros to get citekey, datum and _? I would be happy to clean this up a
> bit, add DOI/ISBN/URL-functionality, documentation and prepare a bug
> report/patch.
>
> (defmacro org-cite-basic-follow--citekey ()
> '(org-element-property :key (car (oref (transient-prefix-object) scope))))
>
> (defmacro org-cite-basic-follow--datum ()
> '(car (oref (transient-prefix-object) scope)))
>
> (defmacro org-cite-basic-follow--_ ()
> '(cadr (oref (transient-prefix-object) scope)))
>
> (transient-define-prefix org-cite-basic-follow (datum _)
> "How should we follow references?"
> [["Open"
> ("b" "bibliography entry"
> (lambda ()
> (interactive)
> (org-cite-basic-goto
> (org-cite-basic-follow--datum)
> (org-cite-basic-follow--_))))]
This looks way too complicated.
Is there an easier way to access transient prefix command arguments from
suffixes? Maybe something provided by transient itself?
--
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>