🎉 Bind K and J to emoji picker

LinusU / emoji-commit

44 stars 6 forks source link

Closed albinekb closed 6 years ago

albinekb commented 6 years ago

Fix #4

Two potential improvements:

Key::Char('k') will only match lowercase k, so if you have caps lock on you can't navigate, feature or bug? 🤔
Should i make up/down selected a function and share it between all the keys that do the same thing or is this good enough?

Both are fixed 👍

LinusU commented 6 years ago

Awesome 👏

How about adding two lines more, with Key::Char('K') and Key::Char('L')?

LinusU commented 6 years ago

Perfect!