88187 - [RFE] navigation keys to move focus to invisible recipients in a long list

Status: VERIFIED WORKSFORME

(SeaMonkey :: MailNews: Message Display, enhancement, P1)

Description

[Michel Poleur]

STEPS TO REPRODUCE

1. Compose a new message
2. Enter several recipients until a scrollbar appears
3. Try to move focus to the first recipient you entered (Shift-TAB)

RESULTS

It's possible to move focus to the first *visible* recipient but not to the
first recipient in the list.

Comment 1

[Keyser Sose]

Marking NEW.

Comment 2

[scottputterman]

reassigning to varada.

Comment 3

[scottputterman]

reassigning to ssu.

Comment 4

[Sean Su]

Attachment: patch v1.0

Comment 5

[racham]

Comment on attachment 68916 [details] [diff] [review]
patch v1.0

r=bhuvan.

As far as autocomplete changes concerned, I think  you certainly need to get
reviews/approvals to make sure that your changes are in line with their design
and has no regressions.
Adding Hewitt, Blake to the cc list. Please any other engineers who are well
versed with this code.

Comment 6

[Joe Hewitt (gone)]

I'd prefer having to add the property disableMenuPopupViaUpDownKeys if possible,
and I think it is possible.  What we need to do is add a capturing keypress
handler that cancels the event bubbling if the up/down key is pressed.

Comment 7

[Sean Su]

> I'd prefer having to add the property disableMenuPopupViaUpDownKeys if
> possible, and I think it is possible.

I don't quite follow.  The patch does have the a new
disableMenuPopupViaUpDownKeys property.

> What we need to do is add a capturing keypress
> handler that cancels the event bubbling if the up/down key is pressed.

I don't know if that is exactly what is needed because we are still in need to
arrow up/down the autocomplete popup list (when shown) as well as ignore the
arrow up/down when the autocomplete popup list is not shown.  What I mean is
that I thought this was all part of the same event.  Woudln't preventing the
event from bubbling up kill the former?

Comment 8

[Michael Buckland]

Discussed at mail news bug meeting.  Decided to RTM this bug, but we think it
might be fixed already.  Is this bug fixed?

Comment 9

[scottputterman]

this worksforme using the PR1 build which is almost identical to RC2.

Comment 10

[esther]

This has been working for months now, verified.

Comment 11

[Brant Gurganus]

[RFE] is deprecated in favor of severity: enhancement.  They have the same meaning.