Skip to content

Fix to be able to change the active line#186

Merged
mattn merged 1 commit intomasterfrom
fix-cursor-move-with-nolim
Mar 1, 2016
Merged

Fix to be able to change the active line#186
mattn merged 1 commit intomasterfrom
fix-cursor-move-with-nolim

Conversation

@tacahiroy
Copy link
Copy Markdown
Member

The problem is the cursor position is reset to the top forcibly
if nolim is set to 1 when you tried to current line.
nolim

The problem is the cursor position is reset to the top forcibly
when you tried to change the active line to the next if nolim is set
mattn added a commit that referenced this pull request Mar 1, 2016
Fix to be able to change the active line
@mattn mattn merged commit b34c940 into master Mar 1, 2016
@mattn mattn deleted the fix-cursor-move-with-nolim branch March 1, 2016 06:22
@mattn
Copy link
Copy Markdown
Member

mattn commented Mar 1, 2016

Thanks

mattn added a commit that referenced this pull request Mar 1, 2016
Because this commit change behavior of initial cursor location
mattn added a commit that referenced this pull request Mar 1, 2016
@mattn
Copy link
Copy Markdown
Member

mattn commented Mar 1, 2016

Sorry, I merged this but reverted. Because this change original behavior. Before merging, I can see the initial cursor location is bottom. But after merged, it move-up to top.

@tacahiroy
Copy link
Copy Markdown
Member Author

ah right. I'll take a look

mattn added a commit that referenced this pull request Mar 9, 2016
Use winsaveview(), winrestview() instead

Fixes #186
@tacahiroy
Copy link
Copy Markdown
Member Author

This issue seemed to be fixed at some point. All good for now

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants