124310 - Need new property to autocomplete for use in mail compose window

Status: RESOLVED FIXED

(SeaMonkey :: Autocomplete, defect)

Description

[Sean Su]

The mail compose window needs to be able to have keyboard navigation (arrow keys
up/down) in the address fields.  However, the autocomplete feature takes over
the arrow keys and prevent proper keyboard navigation between the address fields.

The arrow keys currently bring open the autocomplete menulist.  We need a
property to suppress that feature.

Comment 1

[Sean Su]

I have a fix for this as part of the patch to bug 88187.  reassigning this bug
to myself.

Comment 2

[Sean Su]

The patch to fix this that is attached to bug 88187, appearently is not quite
the correct fix (eventhough it works).

setting target milestone to mozilla1.0 and reassigning to Hewitt, given his
comment in bug 88187.

Comment 3

[Jaime Rodriguez, Jr.]

nsbeta1- per ADT triage.

Comment 4

[Peter Trudelle]

->access, cc aaronl for section 508 implications.

Comment 5

[Aaron Leventhal]

I'm still able to do what I want by pressing enter to accept the current entry
or escape to reject it.

Then I can press tab or shift+tab to move between address fields.

Comment 6

[Peter Trudelle]

Mass-moving all Navigator team 1.0 nsbeta1- bugs to 1.1

Comment 7

[Vedran Miletic]

retargeting

Comment 8

[timeless]

you might be able to use tab, but some people expect up/down to work when
there's no autocomplete needed.

Comment 9

[[e-mail]]

Well, we need to fix autocomplete not to eat up/down keys while the popup is
closed (unless it's configured to open the popup), then we can just catch the
up/down keys to focus the previous/next input fields. Doing it for the popups is
currently impossible due to an xbl/popup frame interaction issue.

Comment 10

[[e-mail]]

Attachment: Proposed patch

Comment 11

[David :Bienvenu]

Comment on attachment 134561 [details] [diff] [review]
Proposed patch

I'll let Scott look at this since he's been doing some auto-complete stuff. It
looks OK to me, however.

Comment 12

[Scott MacGregor]

Comment on attachment 134561 [details] [diff] [review]
Proposed patch

looks good to me.

Comment 13

[[e-mail]]

Fix checked in.