Skip to content
This repository has been archived by the owner on Sep 28, 2022. It is now read-only.

Do not update when a special buffer is selected #63

Merged
merged 1 commit into from
Sep 20, 2016
Merged

Do not update when a special buffer is selected #63

merged 1 commit into from
Sep 20, 2016

Conversation

svanharmelen
Copy link
Contributor

And “special” here, means anything else that a normal buffer (a file).

Additionally make sure we reset the alternate window as well so we can keep using that as expected.

For details why this fix is needed, please see this issue ctrlpvim/ctrlp.vim#235 and the comments (mainly the last 5 or so) of this PR ctrlpvim/ctrlp.vim#303

And “special” here, means anything else that a normal buffer (a file).

Additionaly make sure we reset the alternate window as well so we can keep using that as expected.

For details why this fix is needed, please see this issue ctrlpvim/ctrlp.vim#235 and the comments (mainly the last 5 or so) of this PR ctrlpvim/ctrlp.vim#303
@svanharmelen
Copy link
Contributor Author

@Xuyuanp could you please have a look at this one and merge it if you are OK with it? It fixes a long standing issue with the CtrlP plugin (see the issue and the PR linked to in the comment of this PR) which turns out to be caused by this plugin.

If you have any thoughts or comments please let me know so we can work towards a solution that can be merged.

Thx!

@Xuyuanp Xuyuanp merged commit 1d9e187 into Xuyuanp:master Sep 20, 2016
@Xuyuanp
Copy link
Owner

Xuyuanp commented Sep 20, 2016

Looks good to me.

Thank you.

@mattn
Copy link

mattn commented Sep 20, 2016

Nice work. :)

@svanharmelen
Copy link
Contributor Author

@mattn thx... So the issue reported with CtrlP is really fixed now. Thanks for your help and time as well 😉

@Xuyuanp
Copy link
Owner

Xuyuanp commented Sep 21, 2016

@svanharmelen Hi, what's the software you use to record screen?

@svanharmelen
Copy link
Contributor Author

That's Giphy Capture: http://giphy.com/apps#giphycapture

It's only available for OSX/macOS...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants