Re: [O] PATCH: follow timestamp link also in heading

2012-05-25 Thread Bastien
Samuel Wales samolog...@gmail.com writes:

 Perhaps we should make RET do this also when RET follows links.

Indeed, that's now the case.

Thanks!

-- 
 Bastien



Re: [O] PATCH: follow timestamp link also in heading

2012-05-24 Thread Bastien
Hi Ingo,

Ingo Lohmar i.loh...@gmail.com writes:

 --- a/lisp/org.el
 +++ b/lisp/org.el
 @@ -9606,6 +9606,7 @@
(org-remove-occur-highlights nil nil t)
(cond
 ((and (org-at-heading-p)
 +(not (org-at-timestamp-p t))
  (not (org-in-regexp
(concat org-plain-link-re \\|
org-bracket-link-regexp \\|

Applied in master, thanks.

-- 
 Bastien



Re: [O] PATCH: follow timestamp link also in heading

2012-05-24 Thread Samuel Wales
Thanks for this.  I use inactive timestamps in headings all the time
and sort by them.  It offers excellent usability for finding a contact
-- just scan the dates in order.

I had not noticed the /lack/ of the feature, because I didn't notice
its /presence/ in non-mouse.

I use RET to follow links.

Perhaps we should make RET do this also when RET follows links.

Samuel

-- 
The Kafka Pandemic: http://thekafkapandemic.blogspot.com



[O] PATCH: follow timestamp link also in heading

2012-05-23 Thread Ingo Lohmar
Hi there,

I stumbled upon the functionality to open the agenda for a timestamp
using org-open-at-point.  That is really sweet, but as it turns out, it
does not work in a headline (as of 7.8.10, with no changes to the
relevant code since then).  I cannot see a good reason for this, and it
is a one-line patch to fix it.  Sorry for not using the development
version proper, hence just a unified diff (below).

It would be great if this could be included!
Ingo



--- a/lisp/org.el
+++ b/lisp/org.el
@@ -9606,6 +9606,7 @@
   (org-remove-occur-highlights nil nil t)
   (cond
((and (org-at-heading-p)
+(not (org-at-timestamp-p))
 (not (org-in-regexp
   (concat org-plain-link-re \\|
   org-bracket-link-regexp \\|



Re: [O] PATCH: follow timestamp link also in heading

2012-05-23 Thread Ingo Lohmar
Dammit.  Should work (as in normal text) with inactive timestamps as
well!  See below.

--- a/lisp/org.el
+++ b/lisp/org.el
@@ -9606,6 +9606,7 @@
   (org-remove-occur-highlights nil nil t)
   (cond
((and (org-at-heading-p)
+(not (org-at-timestamp-p t))
 (not (org-in-regexp
   (concat org-plain-link-re \\|
   org-bracket-link-regexp \\|