emacs-orgmode
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [O] [PATCH] lisp/org.el (org-open-at-point): prioritize link


From: Nicolas Goaziou
Subject: Re: [O] [PATCH] lisp/org.el (org-open-at-point): prioritize link
Date: Thu, 27 Feb 2014 23:40:00 +0100

Hello,

Oleh <address@hidden> writes:

> Can someone check this patch and commit it.
> It allows to open links that look like this:
>
>     [[id:164552ee-b92d-4af6-9570-60d039876fac][=foobar=]]

I applied a generalization of your patch. Thank you.

While I'm at it, here are a few comments.

> +        (parent (plist-get (cadr context) :parent))

This is too low level. It should really be:

  (parent (org-element-property :parent context))

> +        (context (if (eq (car parent) 'link) parent context))

Same here. The preferred syntax is:

  (context (if (eq (org-element-type parent) 'link) parent context))


Regards,

-- 
Nicolas Goaziou



reply via email to

[Prev in Thread] Current Thread [Next in Thread]