50329 - Autocomplete should fire on backspace

Status: VERIFIED FIXED

(MailNews Core :: Composition, defect, P2)

Description

[Phil Peterson]

Using 2000-08-25-08 commercial build on NT

1. Bring up compose window
2. Type a common name which will generate a lot of autocomplete hits, like Bob
3. Once you have the menu of choices, type another letter or two from one of the
hits, to narrow it down
4. Now backspace back to just Bob

Expected: list of hits, as when I typed Bob in the first place
Actual: no hits shown - autocomplete seems to fire if you type actual
characters, but not if you type backspace

cc'ing radha since the URL bar autocomplete has the same behavior.

Comment 1

[Phil Peterson]

Adding 4xp and polish keywords

Comment 2

[Jean-Francois Ducarroz]

you are right, my mistake. Should be very simple to fix...

Comment 3

[Jean-Francois Ducarroz]

here is the fix:

Index: autocomplete.xml
===================================================================
RCS file: 
/cvsroot/mozilla/xpfe/components/autocomplete/resources/content/autocomplete.xml
,v
retrieving revision 1.12
diff -u -r1.12 autocomplete.xml
--- autocomplete.xml    2000/08/24 03:55:09     1.12
+++ autocomplete.xml    2000/08/25 23:42:04
@@ -355,17 +355,6 @@

                                switch (event.keyCode)
                                {
-                                       case 8: /*vk_back*/
-                                       case 46: /*vk_delete*/
-                                   if (me.autoCompleteTimer) {
-                                           clearTimeout(me.autoCompleteTimer);
-                                           me.autoCompleteTimer = 0;
-                                   }
-                                               
me.privatefunc.closePopupMenu(me);
-            if (me.lastResults != null)
-                                                 
me.lastResults.searchString='';
-                                               return;
-
           case 9: /*vk_tab*/
                                                if (popup)
               me.privatefunc.closePopupMenu(me);

Comment 4

[lchiang]

+, P2 per mail triage

Comment 5

[Jean-Francois Ducarroz]

fixed and checked in

Comment 6

[esther]

Using build 2000-06 on win98, mac and linux this is fixed.  Verified