Closed
Bug 14856
Opened 25 years ago
Closed 25 years ago
gtk generates incorrect key events for keyrepeat
Categories
(Core :: DOM: UI Events & Focus Handling, defect, P3)
Tracking
()
VERIFIED
FIXED
M16
People
(Reporter: Brade, Assigned: blizzard)
References
Details
(Keywords: platform-parity)
Attachments
(2 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
GTK currently generates extra key down and key up events for each repeated key
press. Instead, the event chain should look like:
keydown, keypress, keypress, keypress, keypress..., keyup
This is per the DOM specification and 4.x compatibility: http://www.w3.org/TR/
WD-DOM-Level-2/events.html#Events-eventgroupings-keyevents
Comment 1•25 years ago
|
||
Looks like gtk is sending key release events even when no key was released; I
wonder if we'll have to go to xlib in order to suppress the fictitious events?
Or does gtk have some way of handling this? If we could avoid getting the key
release events, then the handle_key_press_event code could suppress the KeyDown
event in the case where we already had a KeyDown for that keycode and haven't
gotten a KeyUp yet, or something to that effect.
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M11
Comment 2•25 years ago
|
||
mass-moving most m11 bugs to m12
Comment 3•25 years ago
|
||
Mass-moving non-PDT+ bugs to M13
Comment 4•25 years ago
|
||
does it still do this now that we are using superwin and generating our own key
events?
Comment 5•25 years ago
|
||
Yes, it still happens. If you hold a key down, we still get a sequence of key
down, key press, key up, key down, key press, key up, etc. as long as the key is
held down.
Updated•25 years ago
|
Target Milestone: M13 → M15
Updated•25 years ago
|
Summary: [pp] gtk generates incorrect key events for keyrepeat → gtk generates incorrect key events for keyrepeat
Comment 6•25 years ago
|
||
blizzard, could you please look at this?
Assignee: pavlov → blizzard
Status: ASSIGNED → NEW
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 8•25 years ago
|
||
Assignee | ||
Comment 9•25 years ago
|
||
Assignee | ||
Comment 10•25 years ago
|
||
r=syd, fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 11•25 years ago
|
||
Please ignore the spam. Changing address.
Assignee: blizzard → blizzard
Status: RESOLVED → NEW
Assignee | ||
Comment 12•25 years ago
|
||
bustage from my reassign
Status: NEW → RESOLVED
Closed: 25 years ago → 25 years ago
Comment 13•25 years ago
|
||
Mass update: changing qacontact to ckritzer@netscape.com
QA Contact: janc → ckritzer
Assignee | ||
Comment 14•25 years ago
|
||
This bug is back from the checkin that fixed bug #46901.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 15•25 years ago
|
||
Re-checked in.
Status: REOPENED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → FIXED
Comment 17•24 years ago
|
||
Reassigning QA Contact for all open and unverified bugs previously under Lorca's
care to Gerardo as per phone conversation this morning.
QA Contact: lorca → gerardok
Updated•6 years ago
|
Component: Event Handling → User events and focus handling
You need to log in
before you can comment on or make changes to this bug.
Description
•