[dev] [sandy] Undo, redo and repeat

2014-07-15 Thread Rafa Garcia Gallego
Hi, With Dimitris' recent vi(m) bindings came a bug. BUG: operations added via multiplication appear as a set of unchained single Undo(s). They should be chained so they are undone / redone together. Which made me realize: TODO: need to extend the repeat framework (i.e. the '.' command) to

Re: [dev] [sandy] Undo, redo and repeat

2014-07-15 Thread Dimitris Zervas
On July 15, 2014 11:20:44 AM EEST, Rafa Garcia Gallego rafael.garcia.gall...@gmail.com wrote: Hi, With Dimitris' recent vi(m) bindings came a bug. BUG: operations added via multiplication appear as a set of unchained single Undo(s). They should be chained so they are undone / redone together.

Re: [dev] [sandy] Undo, redo and repeat

2014-07-15 Thread Carlos Torres
Hey, On Tue, Jul 15, 2014 at 4:20 AM, Rafa Garcia Gallego rafael.garcia.gall...@gmail.com wrote: TODO: need to extend the repeat framework (i.e. the '.' command) to store operations, multiplication and their parameters. This reminds me of the record option in vim with qkeytypeStuffsAndEditq

Re: [dev] [sandy] Undo, redo and repeat

2014-07-15 Thread Markus Teich
Carlos Torres wrote: i never got the . command to do much other than repeat the latest insert or delete...(probably me). Heyho, I find it quite handy as an easy way of interactive search and replace. Instead of „:%s/bla/blub/gc“ I can just „*“ on the word, „cw“ or „s5“ once and then hit „n“

Re: [dev] [sandy] Undo, redo and repeat

2014-07-15 Thread Silvan Jegen
On Tue, Jul 15, 2014 at 06:10:57PM +0200, Markus Teich wrote: Carlos Torres wrote: i never got the . command to do much other than repeat the latest insert or delete...(probably me). Heyho, I find it quite handy as an easy way of interactive search and replace. Instead of

Re: [dev] [sandy] Undo, redo and repeat

2014-07-15 Thread Markus Teich
Rafa Garcia Gallego wrote: One thing I like from sandy is that it sets the selection when finding (instead of highlighting the term in another silly way). Maybe we can keep this in sandy-vi, so that: /fooRETcbarESC Works as you expect, despite not being 100% vi(m) compatible. Heyho,