Closed Bug 327101 Opened 17 years ago Closed 16 years ago

Ctrl+Left should close autocomplete


(Firefox :: Address Bar, defect, P4)

2.0 Branch



Firefox 2 beta1


(Reporter: jruderman, Assigned: Gavin)



(Keywords: dataloss, fixed1.8.1)


(1 file, 1 obsolete file)

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, 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.
Flags: blocking-firefox2?
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+
Attached patch wip (diff -w) (obsolete) — Splinter Review
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 →
Target Milestone: --- → Firefox 2 beta1
OS: Windows XP → All
Priority: -- → P4
Hardware: PC → All
Version: Trunk → 2.0 Branch
See also Bug 337695, possible dupe.
Whiteboard: [swag: 1d]
*** Bug 340377 has been marked as a duplicate of this bug. ***
*** Bug 337695 has been marked as a duplicate of this bug. ***
Attached patch patchSplinter Review
Attachment #213544 - Attachment is obsolete: true
Attachment #226066 - Flags: review?(mconnor)
Attachment #226066 - Flags: approval-branch-1.8.1?(mconnor)
Whiteboard: [swag: 1d] → [patch-r?]
Comment on attachment 226066 [details] [diff] [review]

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
Attachment #226066 - Flags: review?(mconnor)
Attachment #226066 - Flags: review+
Attachment #226066 - Flags: approval-branch-1.8.1?(mconnor)
Attachment #226066 - Flags: approval-branch-1.8.1+
Whiteboard: [patch-r?] → [checkin needed+]
mozilla/toolkit/content/widgets/autocomplete.xml 	1.65
Closed: 16 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Whiteboard: [checkin needed+]
You need to log in before you can comment on or make changes to this bug.