BoostNote-App icon indicating copy to clipboard operation
BoostNote-App copied to clipboard

Collapsing left panel: jumping icon and shortcut not working

Open WeberAlex opened this issue 4 years ago • 8 comments

Environment

Boost Note Web App 1.50.4 Windows 10 / Chrome Chrome OS

That's a follow up of #1178. First off, 1.50.4 #1209 is great release 👍 and a big thank you to @Komediruzecki for the continual support.

Issues

Jumping icon

When the mouse pointer is positioned over the icon with the arrow, the icon jumps. I can reproduce it 100% of the time on Windows/Chrome and Chrome OS. Not too surprisingly since this is the same browser.

https://user-images.githubusercontent.com/28292190/134092771-45e29023-a9d0-48d6-ad75-c638e2977b9f.mp4

Shortcut not working

The shortcut Ctrl + Shift + 0 is not working. I have tried with different key combination FN Alt not to avail.

The shortcut Ctrl + , to edit the custom CSS of the preview does work!

My keyboard is a Japanese logicool K240

WeberAlex avatar Sep 21 '21 00:09 WeberAlex

Hi, WeberAlex,

I can reproduce the jumping icon issue as well, will be fixed soon.

The shortcut is working for me, but you need to be careful what key combination triggers which key.

For example Ctrl+Shift+0 won't actually trigger that sequence on some keyboards, but Ctrl+Shift+numpad0 might.

Can you check here: https://keycode.info/ if the keypress you are pressing actually triggers keycode for 0, having a shift key in combination could trigger a second set of keys on regular numbers.

And also try shortcuts with Numpad zero. I know some keyboards don't have Numpad keys it, but typically there is an emulation for it (i.e. to bind it somehow)

Komediruzecki avatar Sep 21 '21 14:09 Komediruzecki

@WeberAlex Seems like we should change the default key binding for Windows. Do you have any preferences?

Rokt33r avatar Sep 22 '21 08:09 Rokt33r

@Rokt33r Ctrl + Shift + X. It is not already attributed to Chrome, and the X key does not change regardless of the keyboard layout (fr, ge, us, etc.)

WeberAlex avatar Sep 22 '21 09:09 WeberAlex

I would suggest Alt+1 or similar, it is versatile enough and it's used by IntelliJ IDEs as the default one. Any combination using Control and Shift might be problematic, especially when used with numbers. We can customize it later when we introduce a keymap for it.

Komediruzecki avatar Sep 22 '21 12:09 Komediruzecki

That's good with me.

WeberAlex avatar Sep 23 '21 04:09 WeberAlex

Alt + 1 calls the files folder in Chrome OS, Alt + 2 , Alt + 3 and Alt + 4 call other functions. Please, disregard my previous comment, Alt + 1 would not be a good idea. I use an Asus Flip C302. Alt + 5 & 6 are available on this model, but I wonder is Ctrl + Shift + Z or X is not a safer bet?

WeberAlex avatar Sep 26 '21 05:09 WeberAlex

It's hard to find something that is not used by some OS, Alt+1 is suitable for the desktop app but not for a web app, since in general browsers bind it to change of tabs with numbers (Chrome, Firefox, Alt+1 first tab, Alt+2 second, etc.).

That's why the best is to have a customizable keymap so users can set it to an appropriate shortcut for themselves.

Komediruzecki avatar Sep 26 '21 07:09 Komediruzecki

How about modifying the shortcut Ctrl + Shift + 0 at first, until releasing the customizable keymap feature?

I can give a list of shortcuts on Windows 10/Chrome that are available.

WeberAlex avatar Sep 29 '21 10:09 WeberAlex