Closed
Bug 865565
Opened 11 years ago
Closed 11 years ago
D3E KeyboardEvent.key values of IME related keys should be considered with keyboard layout locale
Categories
(Core :: Widget: Win32, defect)
Tracking
()
RESOLVED
FIXED
mozilla25
People
(Reporter: masayuki, Assigned: masayuki)
References
Details
(Keywords: dev-doc-complete, inputmethod, intl)
Attachments
(1 file)
Refer current keyboard layout locale at mapping from native keycode to D3E key name index on Windows
11.33 KB,
patch
|
jimm
:
review+
smaug
:
review+
|
Details | Diff | Splinter Review |
On Windows, some virtual keycode values for IME related keys are shared between Japanese keyboard layout and Korean keyboard layout. We should use different key values for them by checking the keyboard layout locale. Additionally, some keys for IME cause OEM virtual keycodes. They should also be mapped with keyboard layout locale.
Assignee | ||
Updated•11 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•11 years ago
|
||
See comment 0 for the detail.
Attachment #768199 -
Flags: review?(jmathies)
Attachment #768199 -
Flags: review?(bugs)
Comment 2•11 years ago
|
||
Comment on attachment 768199 [details] [diff] [review] Refer current keyboard layout locale at mapping from native keycode to D3E key name index on Windows I assume this makes us behave like IE does.
Attachment #768199 -
Flags: review?(bugs) → review+
Assignee | ||
Comment 3•11 years ago
|
||
> I assume this makes us behave like IE does.
IE 10 doesn't support the special keys of Korean keyboard. They are mapped to "Unindentified".
Additionally, "FullWidth", "HalfWidth", "Alphanumeric", "Hiragana" and "Katakana" of Japanese keyboard layout are mapped to "Unidentified".
Finally, IE 10 maps "KanaMode" to "Attn" because the virtual keycode VK_ATTN is used for the Kana Mode/Roman Character mode key. I believe that this is a bug of IE because the key isn't actual "attention" key.
So, our implementation will advance one more step than IE 10.
Updated•11 years ago
|
Attachment #768199 -
Flags: review?(jmathies) → review+
Assignee | ||
Comment 4•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/b5fddca814e0
Comment 5•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/b5fddca814e0
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
Assignee | ||
Comment 6•11 years ago
|
||
https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent#Key_names_and_Char_values
Keywords: dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•