nib icon indicating copy to clipboard operation
nib copied to clipboard

Modifier-key detection only works for Windows and macOS

Open aried3r opened this issue 5 years ago • 1 comments
trafficstars

Hey!

I believe this check https://github.com/nib-edit/Nib/blob/8cc08f1b9e41c8bec5b6e170f9fc05ec60dee4f1/packages/core/src/utils/format-keymap.js#L9 should probably be the inverse, if it's a Mac, the cmd key symbol should be shown, otherwise a circumflex or Ctrl. This also covers all other OS', since macOS is the only one with a cmd key I believe.

On mobile, I'd leave out the shortcuts completely, but I'm not sure how feasible detection is, especially that the iPad Pro keyboard has a command key.

WDYT?

aried3r avatar Jan 21 '20 18:01 aried3r

This definitely can be improved, I will take time to work on it.

jpuri avatar Jan 22 '20 12:01 jpuri