Open
Bug 825337
Opened 12 years ago
Updated 2 years ago
Separate ModifierKeyState from KeyboardLayout
Categories
(Core :: Widget: Win32, defect)
Core
Widget: Win32
Tracking
()
NEW
People
(Reporter: TimAbraldes, Unassigned)
Details
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
ModifierKeyState is used in MetroInput.cpp, WinMouseScrollHandler.cpp, and nsNativeDragTarget.cpp, but none of those files use the other classes in KeyboardLayout.h
It should be possible to safely separate ModifierKeyState from KeyboardLayout
Comment 1•12 years ago
|
||
Hmm, I don't understand why we need to get rid of KeyboardLayout from MetroWidget. I checked MetroInput::InitializeVirtualKeyMap() now. Then, I was surprised. That doesn't care non-US keyboard layout. DOM keyCode cannot be simply mapped from only virtual keycode value. See bug 630810. I think that MetroWidget should use KeyboardLayout if it's possible.
Reporter | ||
Comment 2•12 years ago
|
||
(In reply to Masayuki Nakano (:masayuki) (Mozilla Japan) from comment #1)
> Hmm, I don't understand why we need to get rid of KeyboardLayout from
> MetroWidget. I checked MetroInput::InitializeVirtualKeyMap() now. Then, I
> was surprised. That doesn't care non-US keyboard layout. DOM keyCode cannot
> be simply mapped from only virtual keycode value. See bug 630810. I think
> that MetroWidget should use KeyboardLayout if it's possible.
This bug is about separating ModifierKeyState from KeyboardLayout. I've filed bug 843236 so that we can discuss MetroWidget's use of KeyboardLayout.
Additionally, this bug is more about "organization and neatness" than about real improvements, so I'm un-assigning myself since I should be working on more important metro work :)
Assignee: tabraldes → nobody
Status: ASSIGNED → NEW
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•