Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upivy.el (ivy-next-line): Fix wraparound at end of candidates #118
Conversation
abo-abo
merged commit 856e1bd
into
abo-abo:master
May 24, 2015
1 check passed
continuous-integration/travis-ci/pr
The Travis CI build passed
Details
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment
Hide comment
|
Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
ilohmar commentedMay 24, 2015
The
whenclause previously ran even when the updatedivy-indexwas the proper last candidate(1- ivy--length). When not wrapping, the code still worked, sinceivy-indexis explicitly set again. Withivy-wrapt, the choice was immediately set to the first candidate, so you could not reach the last candidate by going down.Fixed, symmetric to
ivy-previous-line.