Re: [O] [Prelim. patch] extend org-meta-return to keywords

2014-11-22 Thread Thierry Banel
M-RET working as expected is appealing.
Your patch makes M-RET moreuseful than it is now.
I vote +1.

There are a few issues thought.
I guess they can be worked out.

For instance:
   #+HEA|DER: :var v=hello (cursor is the pipe |)
ischanged into
   #+HEA
   #+HEADER: |DER: :var v=hello
which is pretty useless

Also if #+HEADER: is immediately followed by a #+BEGIN_SRC,
the patch (for an unknown reason) is ignored,
and the old behavior applies.

Thanks for this
Thierry


Le 19/11/2014 15:41, Rasmus a écrit :
 Hi,

 Something I have wanted for a while is to have M-RET work as
 expected(?)  on keyword lines such as #+LATEX_HEADER, #+CAPTION etc.

 An animation to illustrate, where '|' is point:

 #+CAPTION: this is a |long caption

 #   click M-RET ⇒ 

 #+CAPTION: this is a 
 #+CAPTION: |long caption

 Would anyone else like this?

 Attached is a quick patch that works surprisingly well.  I would work
 more on it if you guys agree this would be useful.  It probably need
 much more work for corner-cases (any ideas what these are)?

 Thanks,
 Rasmus





Re: [O] [Prelim. patch] extend org-meta-return to keywords

2014-11-22 Thread Rasmus
Hi Thierry,

Thanks for the feedback.
I posted an updated patch under the subject:

  [patch] extend org-meta-return to keywords

where Nicolas commented.

Thierry Banel tbanelweb...@free.fr writes:

 M-RET working as expected is appealing.
 Your patch makes M-RET moreuseful than it is now.
 I vote +1.

 There are a few issues thought.
 I guess they can be worked out.

 For instance:
#+HEA|DER: :var v=hello (cursor is the pipe |)
 ischanged into
#+HEA
#+HEADER: |DER: :var v=hello
 which is pretty useless

Thanks.  I'm pretty sure it's fixed in the version from last night.

 Also if #+HEADER: is immediately followed by a #+BEGIN_SRC,
 the patch (for an unknown reason) is ignored,
 and the old behavior applies.

OK, that's a good test case.  I will look into it.

Thanks,
Rasmus

-- 
Enough with the bla bla!




Re: [O] [Prelim. patch] extend org-meta-return to keywords

2014-11-22 Thread Thierry Banel
Le 22/11/2014 15:31, Rasmus a écrit :

 There are a few issues thought.
 I guess they can be worked out.

 For instance:
#+HEA|DER: :var v=hello (cursor is the pipe |)
 ischanged into
#+HEA
#+HEADER: |DER: :var v=hello
 which is pretty useless
 Thanks.  I'm pretty sure it's fixed in the version from last night.
Sorry, Rasmus, I should have noticed your second patch.
First issue fixed!
Regard, Thierry