On Oct 26, 2010, at 10:21 AM, Nicolas Goaziou wrote:

Hello,

Nathaniel Flath writes:

I think I've fixed the issues brought up with this new patch. Please
let me know what you think.

I've noticed a couple of glitches.

First, you are using

 (> 28 (length struct))

to know when to replace letters by numbers. But (length struct)
doesn't always match list length, so it will lead to errors when
outdenting items.

For example, try outdenting, with its subtree, the item marked with
"<<<" in the list below:

 a) etsiun
 b) etsiun
 c) etsiun
 d) etisun
 e) etsiun
 f) etsiun
 g) etsiun
 h) etsiun
 i) etisun
 j) etsiun
 k) etsiun
 l) etsiun
 m) etsiun
 n) etsiun
    a) Outdent me and my children ! <<<
       a) tersiu
       b) etsiru
       c) estiur
       d) etsnriu
       e) etsiun
       f) etiune
       g) etuirsnet
    b) etsiun
 o) etsiun
 p) etsiun
 q) etsiun
 r) etsiun
 s) etsiun

All the lists will be numbered although they could keep alphabetical
bullets.

Another (lesser) problem is coming from the regexp chosen for bullets,
that is "[0-9A-Za-z]+". I would suggest something alike
"\\(\\(?:[0-9]\\)+\\|[A-Za-z]\\)". At the moment, you can set counter
to [...@a4] and break you list when applying it.


Also, even when the alpha lists are turned off, typing

   a)

and pressing M-RET will show that a) is seen as a list bullet.

Thanks for your work, we are getting closer to an acceptable patch.

- Carsten


Regards,

-- Nicolas

- Carsten




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

Reply via email to