Closed Bug 327101 Opened 17 years ago Closed 16 years ago
Ctrl+Left should close autocomplete
Steps to reproduce on Windows: 1. Start typing into the address bar. 2. Select an autocomplete entry with the Down arrow key. 3. Press Ctrl+(Shift)+Left. 4. Press Delete. (In Firefox 22.214.171.124, press Shift+Delete instead.) Result: autocomplete entry is deleted unexpectedly. Expected: normal text-editing behavior for Delete (delete selection or next character) or Shift+Delete (cut selection). The Left arrow key and Home already close autocomplete, so fixing this bug would also improve consistency. Opt+Left should probably be changed in the same way on Mac (just for consistency), even though this won't be a dataloss issue on Mac once bug 325737 is fixed. This bug was originally mentioned in bug 325737 comment 1 by Gennady Feldman. This can probably be fixed the same way bug 255116 was fixed.
Nominating for blocking Firefox 2. Delete is a more common shortcut than Shift+Delete, so this is likely to be more of a problem now that Delete deletes autocomplete entries instead of Shift+Delete.
Needs a more generalized solution to ensure that any time of action that changes selection or cursor position kills the popup
Flags: blocking-firefox2? → blocking-firefox2+
This seems to fix the case described here, but I've not gone through the different possibilities to make sure this doesn't regress anything. There might be a better fix for this, too. It'd sure be nice if nsIAutoCompleteController used the same key constants as nsIDOMKeyEvent...
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
OS: Windows XP → All
Priority: -- → P4
Hardware: PC → All
Version: Trunk → 2.0 Branch
See also Bug 337695, possible dupe.
*** Bug 340377 has been marked as a duplicate of this bug. ***
*** Bug 337695 has been marked as a duplicate of this bug. ***
Whiteboard: [swag: 1d] → [patch-r?]
Comment on attachment 226066 [details] [diff] [review] patch so, it'd be nice to kill the event and return true after each switch block if cancel is true, since then we skip doing a bunch of additional checks, but its not that critical
Whiteboard: [patch-r?] → [checkin needed+]
mozilla/toolkit/content/widgets/autocomplete.xml 126.96.36.199 mozilla/toolkit/content/widgets/autocomplete.xml 1.65
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed+]
You need to log in before you can comment on or make changes to this bug.