Closed Bug 13962 Opened 25 years ago Closed 25 years ago

[beta][PP]gfx text input fields do not handle forward delete

Categories

(Core :: Layout: Form Controls, defect, P1)

x86
Linux
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: phillip, Assigned: akkzilla)

Details

Overview Description: the location bar and other gfx text input controls do not accept the delete key. instead of deleting the character to the right of the cursor, the character disappears and is replaced by what looks like a space. Steps to Reproduce: 1) open apprunner in linux (GFX on) 2) the location bar should say http://www.mozilla.org (or something) 3) put the cursor in the middle of the word and press the delete key 4) retype the letter you deleted and hit enter. Actual Results: First, the letter you deleted is replaced by a space. then, when you retype it, it is inserted before the space. then, when you hit enter, apprunner does not understand the url. Expected Results: The letter should be deleted, be inserted by you, and mozilla.org should reload. Occurs on: 1999-09-15-09-M10 RedHat Linux 6.0 (GNOME/enlightenment) but NOT on: 1999-09-15-08-M10 WinNT 4.0 sp5 1999-09-15-08-M10 MacOS 8.51
Assignee: karnaze → buster
Reassigning to Steve.
Assignee: buster → akkana
Severity: normal → major
Priority: P3 → P1
Summary: [GFX][PP]gfx text input fields do not handle forward delete → [beta][PP]gfx text input fields do not handle forward delete
Target Milestone: M12
akkana is handling linux key bindings. this must get done for beta, can be done in M12.
Status: NEW → ASSIGNED
This isn't a Linux keybinding issue; it's that the editor key handler handles delete on KeyPress, but everything else on KeyDown, so it tries to insert the delete character after it's already done a delete. It's time to get rid of the KeyDown handling in nsEditorEventListeners.cpp and do everything on KeyPress, or we'll keep having problems like this. I'm going to do that. Cc'ing buster and tague in case one of them has objection to this plan.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Target Milestone: M12 → M11
Fixed. All KeyDown handling in the editor event listeners has been moved to use KeyPress instead.
QA Contact massive update.
Marking VERIFIED FIXED on: - Linux6 2000-02-01-10 Commercial build - Win98 2000-02-01-08 Commercial build - MacOS86 2000-02-01-09 Commercial build
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.