Rudolf Adamkovič <[email protected]> writes:

>> This is getting out of control.
>> Maybe we can use `set-transient-map'?
>
> Just to be clear, Transient is now a go in Org?

set-transient-map has nothing to do with transient.el.
set-transient-map is simply a temporary overriding keymap to be used.
See 23.9 Controlling the Active Keymaps section of Elisp manual.

> -     "Attach a file using copy method.")
> +     "Attach file using copy method...")

We are getting even less grammatical by dropping yet another article (in
addition to already missing the [copy method]) :)
Then, why not even "Attach file via copy..."

> -     "Synchronize the current node with its attachment\n directory, in case \
> -you added attachments yourself.\n")
> +     "Synchronize current heading with attachment directory
> +(reflect externally added/removed attachments)\n")

Maybe "Update :ATTACH: tag to match actual attachment directory contents"

> -     "Set a specific attachment directory for this entry. Sets DIR 
> property.")
> +     "Set attachment directory as DIR property...")
>      ((?S ?\C-S) org-attach-unset-directory

This reads like "Set [existing] attachment directory as DIR property",
which is not what that menu does.

-- 
Ihor Radchenko // yantar92,
Org mode maintainer,
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>

Reply via email to