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

C-g should cancel #65

Open
lolbinarycat opened this issue Sep 4, 2020 · 5 comments
Open

C-g should cancel #65

lolbinarycat opened this issue Sep 4, 2020 · 5 comments

Comments

@lolbinarycat
Copy link

I just installed this, so perhaps I'm missing something, but when I ran the command to test it out, it was a bit annoying to cancel.

First I tried C-g, which didn't work, so I just closed the buffer, but then I was stuck with the selected item from the kill ring. The text couldn't be deleted, so I had to close the buffer entirely to get stuff back in order.

@impaktor
Copy link

impaktor commented Sep 5, 2020

Works for me.

My only config for this part is (which is probably unrelated):

(when (require 'browse-kill-ring nil 'noerror)
  (browse-kill-ring-default-keybindings)
  (setq browse-kill-ring-quit-action 'save-and-restore))

@T-J-Teru
Copy link
Member

T-J-Teru commented Sep 6, 2020

C-g should be bound to browse-kill-ring-quit in the browse kill-ring buffer without requiring any additional setup. To double check this I did emacs -Q -l /path/to/browse-kill-ring.el then placed a few things in the kill-ring before M-x browse-kill-ring. Then in the new buffer that opens I'm able to use either q or C-g to close the buffer.

You could try opening starting browse-kill-ring in your usual setup, then in the browse buffer try C-h b to view the bindings. It's possible that something else is stealing the C-g binding maybe?

@impaktor
Copy link

impaktor commented Sep 6, 2020

...or just: C-h k C-g in the kill-ring

@lolbinarycat
Copy link
Author

C-g is bound to keyboard-quit.
Pressing q works for me, though. After looking back, that keyboard shortcut is actually mentioned in the minibuffer (which is what I was going to suggest).

@ssnnoo
Copy link

ssnnoo commented Jan 7, 2021

C-g this seems to work fine, bound to browse-kill-ring-quit.

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

No branches or pull requests

4 participants