search

OmniSharp / omnisharp-emacs

Troll coworkers - use Emacs at work for csharp!
GNU General Public License v3.0
512 stars 94 forks source link

omnisharp-auto-complete bypasses input-decode-map and key-translation-map #493

Open akeley98 opened 4 years ago

akeley98 commented 4 years ago

I have swapped C-t and C-p in my .emacs file:

(define-key input-decode-map (kbd "C-t") (kbd "C-p")) (define-key input-decode-map (kbd "C-p") (kbd "C-t"))

This works everywhere in emacs (across all major modes that I use, and in term-mode), except in omnisharp-auto-complete. When the menu pops up, only physical Ctrl-p and <up> inputs move the selection up, not Ctrl-t.

You may not agree with my swapping (ctrl-p is hard to input on dvorak keyboards), but as a user, I would appreciate it if omnisharp respects my key remappings as all other emacs packages do.

akeley98 commented 4 years ago

My current workaround is to put

(define-key map "\C-t" 'popup-previous)

in the popup-menu-keymap function in popup.el, then delete popup.elc to force recompile.

Not sure why the popups don't respect key translation (it uses read-key-sequence, which claims to do key translation). May want to report the bug to the maintainers of popup.el instead.