New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 633838 link

Starred by 9 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

Bug report: Incorrect event.key on key event when pressed together with control modifier on custom keyboard layout

Reported by michabar...@gmail.com, Aug 3 2016

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.82 Safari/537.36

Steps to reproduce the problem:
1. Change your keyboard layout to dvorak
2. Monitor keys pressed using: document.addEventListener('keydown', e => console.log(e.key));
3. Press Control+c
4. The console will print: Control\ni

What is the expected behavior?
The console should print: Control\nc

What went wrong?
When using a non-qwerty `event.key` will be set incorrectly to the underlying qwerty key when used together with the Control modifier.

What didn't went wrong?
---

`Alt + key`, `Meta + key`, `Shift + key`, `key` works fine!

`event.keyCode` is also set correctly at any time.

Obviously not only the keydown event is affected.

All chromium shortcuts work correctly and are not affected by this bug.

Did this work before? N/A 

Chrome version: 52.0.2743.82  Channel: dev
OS Version: 4.6.3-300.fc24.x86_64
Flash Version: Shockwave Flash 22.0 r0

I guess this worked correctly in some version of chromium since I didn't always have this issue but it must have been a couple version ago.
 

Comment 1 by kochi@chromium.org, Aug 3 2016

Components: -Blink Blink>Input
Owner: azurewei@chromium.org
Status: Assigned (was: Unconfirmed)
azurewei@, could you take a look at this?
Or if you know someone more appropriate, please reassign.
This bug also got shipped into 53.0.2785.92
Owner: dtapu...@chromium.org
Cc: dtapu...@chromium.org kpschoedel@chromium.org
Components: -Blink>Input IO>Keyboard
Labels: Hotlist-Input-Dev
Owner: chongz@chromium.org
Status: Started (was: Assigned)
 Issue 663339  has been merged into this issue.
It has been like this for me since at least June.  I don't remember if I was on the dev channel back then, but I at some point switched to stable and still have it.

Comment 7 by chongz@chromium.org, Nov 14 2016

Labels: M-56
Status: Fixed (was: Started)
https://crrev.com/2474083002

Comment 8 by chongz@chromium.org, Jan 10 2017

Cc: chongz@chromium.org
 Issue 585241  has been merged into this issue.

Comment 9 by chongz@chromium.org, Jan 10 2017

 Issue 580566  has been merged into this issue.

Sign in to add a comment