olegberman / atom-shortcuts

:airplane: Learn Atom's shortcuts (cheatsheet)
https://atom.io/packages/atom-shortcuts
MIT License
9 stars 13 forks source link

Implement keyboard shortcuts in proper atom api format #1

Closed zenopopovici closed 4 years ago

zenopopovici commented 9 years ago

The keyboard shortcuts are impossible to override right now. This causes clashes with other extensions. Also the keyboard shortcut doesn't appear in the atom keymap list.

olegberman commented 9 years ago

Okay, since I've already got 140 downloads I will try to make it right.

The issue that I was having is that I couldn't figure out how to catch keyupand keydown separately through normal atom keymaps (as it only catches a complete stroke of a shortcut). The convenience of being able to hide the cheatsheet by just releasing a button overdrew the will to make it right at the moment. Only for that reason I've chosen a shortcut that is comfortable for your fingers and probably not very commonly used.

I will look into Atom's API again, maybe I'm missing something. Thanks so much for your interest!

zenopopovici commented 9 years ago

I actually hate it that I do have to keep it pressed, you do need to scan for what you need :) But I guess usage highly differs. The other problem with the current implementation is that it also disregards 3 key commands any shortcut that has the ctr+`+any_other_key combination will open the popup. Which is bad.

You could set a timer for it or just implement a simple toggle.

Great work tough, I'm recommending it to newcomers to Atom.

pbraga commented 9 years ago

+1 for not being able to override the shortcut. In Portuguese keyboards the back-tick need a shift press...so i cannot use this plugin at all.... Do like the idea of a pop up cheatsheet

Rosefalk commented 8 years ago

Can't use this with a Danish keyboard since it requires shift.

witwit commented 8 years ago

Same here, backtick is a bad choice for toggle

olegberman commented 8 years ago

2,628 installs! Definitely will look into this.

nickooolas commented 8 years ago

Everybody got the ctrl + ` for lots others extensions. Would be great to be able to change it. BTW: 3,227 installs!

ghost commented 7 years ago

+1 and 4,948 installs.

ethernal commented 7 years ago

+1 6,815 and counting - we need a proper fork probably. OR a fix. I have no idea how packages for atom are made but... maybe...

BTW ctrl+` is default for console components and for devs/gamers? that's quite standard I think.

progand commented 5 years ago

I've found similar tool https://github.com/jkasky/atom-keybinding-cheatsheet

olegberman commented 5 years ago

Do you guys still need this extension? Vote for the trigger shortcut that you want to use.

P.S. 5 issues / 0 forks?

progand commented 5 years ago

I suggest Ctrl+Alt+/

Frederic-C commented 5 years ago

It displays only by pressing "ctrl", which is super anoying. Unable to use it :/