Closed Bug 672561 Opened 9 years ago Closed 9 years ago

"awDocumentKeyPress is not defined" when opening Mailing List dialog [New List]

Categories

(SeaMonkey :: MailNews: Address Book & Contacts, defect)

defect
Not set
minor

Tracking

(seamonkey2.3 wontfix, seamonkey2.4 fixed)

RESOLVED FIXED
seamonkey2.5
Tracking Status
seamonkey2.3 --- wontfix
seamonkey2.4 --- fixed

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

Details

Attachments

(1 file, 1 obsolete file)

Error:
awDocumentKeyPress is not defined
chrome://messenger/content/addressbook/abMailListDialog.js
Line: 245

TB has a corresponding function here:
http://mxr.mozilla.org/comm-central/source/mail/components/compose/content/addressingWidgetOverlay.js#1049
Attached patch patch (obsolete) — Splinter Review
This should probably go into Aurora then, too.
Assignee: nobody → jh
Status: NEW → ASSIGNED
Attachment #551556 - Flags: review?(neil)
We removed this from the compose window in bug 603101 because it was buggy, but we forgot to check for shared uses in mailnews :-(

Can you also make it call awRecipientKeyPress, so we get the nice up/down behaviour as well as shift+tab? (Sadly the version of awTabFromRecipient in abMailListDialog.js is buggy so we don't get the right tab behaviour. Any idea how (or if) Thunderbird handles this?)

[Ideally we would nuke awDocumentKeyPress and use handleKeyPress instead.]
(In reply to neil@parkwaycc.co.uk from comment #2)
> Can you also make it call awRecipientKeyPress, so we get the nice up/down
> behaviour as well as shift+tab?

I can confirm that using awRecipientKeyPress gives us up/down arrow key support, which is indeed nice, but I also have Shift+Tab working without this patch at all, so what do you mean there (see also next point)?

> (Sadly the version of awTabFromRecipient in
> abMailListDialog.js is buggy so we don't get the right tab behaviour. Any
> idea how (or if) Thunderbird handles this?)

First I'd have to understand what the "right" tab behavior is in your eyes. AFAICS TB behaves like us without the patch.
Attachment #551556 - Attachment is obsolete: true
Attachment #551893 - Flags: review?(neil)
Attachment #551556 - Flags: review?(neil)
(In reply to Jens Hatlak (:InvisibleSmiley) from comment #3)
> First I'd have to understand what the "right" tab behavior is in your eyes.
The "right" tab behaviour is what you get in the Compose window, where "tab" cycles through all the addresses, even when they have scrolled out of view.
Comment on attachment 551893 [details] [diff] [review]
patch v1a [Checkin: comments 6 and 8]

Well, I guess the tab handling is consistent this way, so better than v1.
Attachment #551893 - Flags: review?(neil) → review+
Comment on attachment 551893 [details] [diff] [review]
patch v1a [Checkin: comments 6 and 8]

http://hg.mozilla.org/comm-central/rev/551872168f3a
Attachment #551893 - Attachment description: patch v1a → patch v1a [Checkin: comment 6]
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.5
Comment on attachment 551893 [details] [diff] [review]
patch v1a [Checkin: comments 6 and 8]

This gets rid of an error message and allows using the arrow keys in the Mailing List dialog (again?). Should be safe enough for Aurora so it'll be in 2.4.
Attachment #551893 - Flags: approval-comm-aurora?
Attachment #551893 - Flags: approval-comm-aurora? → approval-comm-aurora+
Comment on attachment 551893 [details] [diff] [review]
patch v1a [Checkin: comments 6 and 8]

http://hg.mozilla.org/releases/comm-aurora/rev/ab422cc8f58c
Attachment #551893 - Attachment description: patch v1a [Checkin: comment 6] → patch v1a [Checkin: comments 6 and 8]
You need to log in before you can comment on or make changes to this bug.