Closed Bug 179617 Opened 22 years ago Closed 22 years ago

Keys ignored editing message recipients (BS & Del)

Categories

(MailNews Core :: Composition, defect)

x86
OS/2
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: mrmazda, Assigned: mkaply)

References

Details

Attachments

(1 file)

2002111108 and 2002111115 OS/2 trunk. To reproduce: 1-Compose a reply message 2-Try to change to, cc, group, etc Actual results: 1-Backspace and delete keys ignored Expected results: 1-Functional backspace and delete keys
-->varada
Assignee: ducarroz → varada
Actually, I checked and this is OS/2 only. Do you have any idea when this started?
Assignee: varada → mkaply
Alan Beagley is the one who noticed and posted on Usenet. No Sunday or Saturday builds to try. The rest since the 5th were all unusable for mail.
So it was working on the 5th?
2002110508 trunk OK. I unzipped it again just to check for you.
Thanks, I really appreciate it. Javier, could you have someone narrow this down. We should have the nightly builds from the 6,7,8 on our internal build machine bwbld116 (there were network problems) Hopefully this will be an easy one.
In 2002111208 trunk I just tried to confirm for additional comments, and the activity meter pegged. What I was doing when the race started was alternating quickly between striking random keys and changing focus with the mouse. I had to kill Mozilla. I selected -> File -> New -> Message, and discovered, that I can select the BCC address text with keyboard or mouse, and Shift-Del will not cut, but cut from edit menu works.
*** Bug 180283 has been marked as a duplicate of this bug. ***
Blocks: 180455
This looks like the same problem that I fixed in a couple of places for linux. You're reusing a key event struct in nsWindow::OnKey. Some flags are set during event dispatch that make this not a good idea. I bet this goes away if you change it to use a new struct. I had to fix this for both key events and context menu events on Linux.
Attached patch Fix (deleted) — Splinter Review
bryner - you rock as usual. Reuse of event was causing the problem. Fix attached.
Fix checked in
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Verified in 2002112209 OS/2 trunk.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: