Automattic / simplenote-electron

Simplenote for Web, Windows, and Linux
https://app.simplenote.com
GNU General Public License v2.0
4.78k stars 557 forks source link

On Russian keyboards, "Б" character adds a tag #2040

Open codebykat opened 4 years ago

codebykat commented 4 years ago

Expected

We should split on commas regardless of keyboard layout

Observed

On Russian keyboards, the "Б" character is apparently the same keycode as the comma on English keyboards so typing it causes a tag to be added.

It's possible this also implies that comma doesn't work to add a tag for Russian keyboards (since it would be a different keycode). I haven't tested that scenario.

User reported in 2896068-zen and reproduced it in Windows 10 and on web.

codebykat commented 4 years ago

Research for later:

https://github.com/aehlke/tag-it/issues/22 https://github.com/aehlke/tag-it/issues/140 https://github.com/aehlke/tag-it/issues/251#issuecomment-39100825

And by the way, on Russan layout, COMMA is a keycode 191 with SHIFT pressed. So pressing this COMMA does not insert new tag.

when i type "ม" in thai language this "ม" is same button "," (comma). i change condition split tag (c.which===b.ui.keyCode.COMMA&&!1===c.shiftKey) to (event.key === ","). i can type "ม"

dmsnell commented 4 years ago

Nice. We should be able to fix this. In the keybindings audit we missed this one - it relies on the old constants for keypress, which is silly. We'll update to use the , - thanks @codebykat for the link to that tag-it PR

rezzap commented 4 years ago

User has provided the following information and recording of the issue to explain the issue further.

Comma in english layout adds tag instantly. Comma in russian layout separated tag to several after Enter pressing (see attached video

Screen Recording 2020-05-01 at 11 56 am

msilbers commented 4 years ago

reported in 3294593-zen

dmsnell commented 4 years ago

@rezzap the video demonstrates another aspect of tagging - tags are separated by a space. this has always been how tags work as far as I can remember. originally it made it easier to add multiple tags from a mobile device.

reginabally commented 4 years ago

Another report in 3333717-zen.

codebykat commented 4 years ago

This should have been fixed by #2334 but I just tested and it was not, so apparently I missed one.

reginabally commented 3 years ago

Reported in 3474956-zen with the small letter b in Russian - "б".

eduardozulian commented 3 years ago

Also reported in 3669598-zen with б

codebykat commented 3 years ago

Also reported in 3853286-zen

eduardozulian commented 3 years ago

Also reported in 4265268-zen

eduardozulian commented 3 years ago

Another: 4320340-zen

pachlava commented 2 years ago

+1 from 4443696-zd-woothemes

CameronMilliken commented 2 years ago

I am using simple note on various devices (android, macos, windows) and would like to report bug in windows version of app

When I am trying to type tag in Russian, which contains letter “б” (like “b” in English) it acts like I am pressing “Enter”. Therefore I cannot type tags containing “б” via app.

«б» letter on Cyrillic keyboard is located where symbols “,” and “<” are placed on US layout, right to “M” key

If I type the whole word elsewhere (notepad, word) and copy it to tags – it works fine

image001

https://woothemes.zendesk.com/agent/tickets/5174559

lromero16 commented 2 years ago

Greetings. On the PC version, the beech "B" in Cyrillic does not work in the tags. Video in attachment.

https://user-images.githubusercontent.com/69362760/179097051-f3e85d5b-5843-4d06-b31c-b9851007c456.mp4

Ticket #5365169