Spun off from bug 92491, which contains a patch for menus only. assume user press 'A', and: 1. the list has neither shortcut key 'A', nor first letter 'A' -- do nothing 2. the list has only one item which has either shortcut key 'A' or first letter 'A' -- select it and do its action 3. the list has more than one item that has either shortcut key 'A' or first letter 'A' -- go circularly within them but do not act them For condition 3, we should also support incremental typing (IE's Favorities does not support).

copy listbox.xml to dist\WIN32_D.OBJ\bin\chrome\toolkit\content\global\bindings (windows) or put it into mozilla/chrome/toolkit.jar (UNIX)

Hi, Jan, Could you review this similar bug?

I tested browser, mailnews, composer, pref, address book. The patch works properly. As far as the autocomplete feature in mail compose, I found that every key typing event is sent to the textbox, so the listbox can not receive any keypress event at all.

Comment on attachment 77791 [details] [diff] [review] patch (with disableKeyNavigation property) r=varga

Rather than .8 seconds, for the keyboard repeat delay on Windows I think we're supposed to use: GetSystemParametersInfo() SPI_GETKEYBOARDDELAY Get settings for delay time of key inputs.

changes: 1. event.charCode >= 32 && event.charCode <= 122 => event.charCode > 0 2. if (event.timeStamp - this._lastKeyTime > 2500) => if (event.timeStamp - this._lastKeyTime > 1000) Jan, can you r= again?

Comment on attachment 78863 [details] [diff] [review] patch (revised according to mpt's comments) r=varga

um. can i get verification on one thing before you guys do this? are you changing the behavior of mail folderpane so that i can't press n to get my next unread message? Kyle: you just ignored aaronl's comments, SRs: please make sure the patch you sr addresses aaronl's comment 5.

Timeless, bug 133366 for tree should answer your questions: - That setting I spoke about in comment #5 is for keyboard repeat rate. - Kyle added a new property called disableKeyNavigation to deal with the mail panes

Comment on attachment 80362 [details] [diff] [review] change nothing, just keep the patch up to date. r=varga

Comment on attachment 80362 [details] [diff] [review] change nothing, just keep the patch up to date. sr=jst

checked into trunk!

This change is causing JS exceptions in the mail/news addressingWidgetOverlay. Please see bug 148010.

verif patch is in.