[UI Events-key] KeyboardEvent.key value of printable keys with Control are usually wrong on macOS

RESOLVED FIXED in Firefox 54

Status

()

defect
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: masayuki, Assigned: masayuki)

Tracking

Trunk
mozilla54
All
macOS
Points:
---

Firefox Tracking Flags

(firefox54 fixed)

Details

Attachments

(1 attachment)

When Control is pressed, KeyboardEvent.key should be computed without Control key (except when Ctrl+Foo causes inputting text). However, as far as I tested, some keys work as expected but especially when Shift key is also pressed, the value is odd. For example, when I press Ctrl+Semicolon, key value is ';' as expected but Ctrl+Shift+Semicolon, key value is also ';', rather than '+' with Japanese Keyboard layout.

I guess that we need to reimplement around here when Ctrl key is pressed.
Assignee: nobody → masayuki
Status: NEW → ASSIGNED
Comment hidden (mozreview-request)

Comment 4

2 years ago
mozreview-review
Comment on attachment 8842258 [details]
Bug 1342865 When Control key is pressed and InsertText() isn't called on macOS, its KeyboardEvent.key value should be characters which are inputted by the key without Control key state

https://reviewboard.mozilla.org/r/116136/#review118118
Attachment #8842258 - Flags: review?(m_kato) → review+

Comment 5

2 years ago
Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/3bee9644cf4c
When Control key is pressed and InsertText() isn't called on macOS, its KeyboardEvent.key value should be characters which are inputted by the key without Control key state r=m_kato

Comment 6

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/3bee9644cf4c
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.