add ac-expand-previous #198

Merged
merged 1 commit into from Jan 16, 2015

Projects

None yet

2 participants

@jpkotta
jpkotta commented Dec 20, 2012

I like to do (define-key ac-complete-mode-map (kbd "<backtab>") 'ac-expand-previous), so that S-TAB moves up the list in case I go past the completion I want.

@m2ym
Member
m2ym commented Dec 21, 2012

Why not ac-previous?

@jpkotta
jpkotta commented Dec 21, 2012

I don't like the undo behavior when using ac-expand + ac-previous. It adds an extra undo boundary.

One cannot go up and down the list endlessly with ac-expand + ac-previous; after doing ac-previous, the next ac-expand chooses a candidate and exits completion. With ac-expand + ac-expand-previous, you can go up and down the list as much as you like.

I like the symmetry of ac-expand + ac-expand-previous; they "work the same way". With ac-expand + ac-previous, going up the completion list is different than going down. I find it more pleasing if going up and down behave the same way.

@m2ym m2ym merged commit a924844 into auto-complete:master Jan 16, 2015

1 check passed

default The Travis build passed
Details
@m2ym
Member
m2ym commented Jan 16, 2015

Okay, I'm convinced.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment