Re: [Orgmode] Remove redundant Select link prompt in C-c C-o

2009-09-08 Thread Carsten Dominik

OK to this change as well.

- Carsten

On Sep 8, 2009, at 1:35 AM, Bastien wrote:


Since the minibuffer already prompts the user to select a link
when C-c C-o triggers a choice, no need to prompt this again in
the temporary window.

Objections?

diff --git a/lisp/org.el b/lisp/org.el
index b58cacc..39ef466 100644
--- a/lisp/org.el
+++ b/lisp/org.el
@@ -8094,7 +8094,6 @@ there is one, offer it as link number zero.
(save-window-excursion
  (delete-other-windows)
  (with-output-to-temp-buffer *Select Link*
-   (princ Select link\n\n)
(mapc (lambda (l)
(if (not (string-match org-bracket-link-regexp l))
(princ (format [%c]  %s\n (incf cnt)

--
Bastien
___
Emacs-orgmode mailing list
Remember: use `Reply All' to send replies to the list.
Emacs-orgmode@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-orgmode




___
Emacs-orgmode mailing list
Remember: use `Reply All' to send replies to the list.
Emacs-orgmode@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-orgmode


Re: [Orgmode] Remove redundant Select link prompt in C-c C-o

2009-09-08 Thread Bastien
Carsten Dominik carsten.domi...@gmail.com writes:

 OK to this change as well.

Done as well.

-- 
 Bastien


___
Emacs-orgmode mailing list
Remember: use `Reply All' to send replies to the list.
Emacs-orgmode@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-orgmode


[Orgmode] Remove redundant Select link prompt in C-c C-o

2009-09-07 Thread Bastien
Since the minibuffer already prompts the user to select a link 
when C-c C-o triggers a choice, no need to prompt this again in 
the temporary window.

Objections?

diff --git a/lisp/org.el b/lisp/org.el
index b58cacc..39ef466 100644
--- a/lisp/org.el
+++ b/lisp/org.el
@@ -8094,7 +8094,6 @@ there is one, offer it as link number zero.
 	(save-window-excursion
 	  (delete-other-windows)
 	  (with-output-to-temp-buffer *Select Link*
-	(princ Select link\n\n)
 	(mapc (lambda (l)
 		(if (not (string-match org-bracket-link-regexp l))
 			(princ (format [%c]  %s\n (incf cnt)

-- 
 Bastien
___
Emacs-orgmode mailing list
Remember: use `Reply All' to send replies to the list.
Emacs-orgmode@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-orgmode