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

Implement C-(S)-tab buffer switch in Ivy layer #14287

Conversation

dalanicolai
Copy link
Contributor

@dalanicolai dalanicolai commented Jan 11, 2021

These keybindings offer a nearly equivalent experience to the common C-tab buffer switching behavior. To me this is still the superior way for switching between previous buffers (more than only one like SPC-tab). Also, I think many other users like this behavior, see:
https://stackoverflow.com/questions/4146574/how-to-properly-configure-ctrl-tab-in-emacs
https://stackoverflow.com/questions/4133181/emacs-tab-between-buffers
https://www.reddit.com/r/emacs/comments/9z4faf/ctrltab_buffer_switching/
(the only downside is that new users maybe will not use SPC b b when they should, but I guess they will).

I am not sure if the keystring used in the last defined keybinding is universal, but this one is working for my keyboard.

As far as I know these bindings do not interfere with any other bindings (i.e. they were all still undefined).

It is just a proposal... no need to merge it if you think it does not fit.

The table was outlined using org-cycle and emacs -Q.

@dalanicolai dalanicolai force-pushed the Implement_ctrl_tab_buffer_switch_in_ivy_layer branch from 16abd04 to c32c562 Compare January 11, 2021 13:01
@lebensterben
Copy link
Contributor

I think this is a nice addition.

dalanicolai added a commit to dalanicolai/spacemacs that referenced this pull request Jan 29, 2021
@dalanicolai dalanicolai force-pushed the Implement_ctrl_tab_buffer_switch_in_ivy_layer branch from c32c562 to 7cadd7b Compare January 29, 2021 10:47
smile13241324 pushed a commit that referenced this pull request Feb 3, 2021
Copy link
Collaborator

@smile13241324 smile13241324 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep this should be merged, I think we have the belated fix for helm already merged.

@smile13241324
Copy link
Collaborator

Merged manually.

@dalanicolai dalanicolai deleted the Implement_ctrl_tab_buffer_switch_in_ivy_layer branch February 15, 2021 10:05
aam-at pushed a commit to aam-at/spacemacs that referenced this pull request Mar 23, 2021
wang-d pushed a commit to wang-d/spacemacs that referenced this pull request Jul 22, 2021
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.

3 participants