Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set buffer-predicate to spacemacs/useful-buffer-p #6574

Closed
wants to merge 1 commit into from

Conversation

bmag
Copy link
Collaborator

@bmag bmag commented Jul 14, 2016

Prevents next-buffer, other-buffer, etc. from choosing useless buffers.
No need for spacemacs/next-useful-buffer, spacemacs/previous-useful-buffer anymore, so I replaced their bindings with next-buffer and previous-buffer.

Fixes spacemacs/alternate-buffer to respect buffer-predicate. It no longer uses evil-alternate-buffer.

When spacemacs-layouts is used, buffer-predicate filters useful buffer that belong to the current layout. For example, SPC b n will not show useless buffers or buffers that don't belong to the current layout.

Caveats:

Fixes #6159 as much as is reasonably possible.

Prevent next-buffer, other-buffer, etc. from choosing useless buffers.
No need for spacemacs/next-useful-buffer,
spacemacs/previous-useful-buffer anymore.

Also fix spacemacs/alternate-buffer to respect buffer-predicate.

When spacemacs-layouts is used, buffer-predicate filters useful buffer
that belong to the current layout.
@TheBB
Copy link
Collaborator

TheBB commented Jul 25, 2016

Thanks! Cherry-picked in develop. You can safely delete your branch.

@TheBB TheBB closed this Jul 25, 2016
@izahn
Copy link
Contributor

izahn commented Jul 26, 2016

This breaks stuff, see #6678. Can we revert until we figure out why and how to fix it?

@TheBB
Copy link
Collaborator

TheBB commented Jul 26, 2016

You mean #6678

@izahn
Copy link
Contributor

izahn commented Jul 26, 2016

Yes #6678 thanks. I edited my previous comment to link to the correct issue.

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

Successfully merging this pull request may close these issues.

4 participants