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

RESOLVED FIXED in seamonkey2.5

Status

SeaMonkey
MailNews: Address Book & Contacts
--
minor
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

Tracking

Trunk
seamonkey2.5

SeaMonkey Tracking Flags

(seamonkey2.3 wontfix, seamonkey2.4 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

7 years ago
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
(Assignee)

Comment 1

7 years ago
Created attachment 551556 [details] [diff] [review]
patch

This should probably go into Aurora then, too.
Assignee: nobody → jh
Status: NEW → ASSIGNED
Attachment #551556 - Flags: review?(neil)

Comment 2

7 years ago
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.]
(Assignee)

Comment 3

7 years ago
Created attachment 551893 [details] [diff] [review]
patch v1a [Checkin: comments 6 and 8]

(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)

Comment 4

7 years ago
(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 5

7 years ago
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+
(Assignee)

Comment 6

7 years ago
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]
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.5
(Assignee)

Comment 7

7 years ago
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?

Updated

7 years ago
Attachment #551893 - Flags: approval-comm-aurora? → approval-comm-aurora+
(Assignee)

Comment 8

7 years ago
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]
(Assignee)

Updated

7 years ago
status-seamonkey2.3: --- → wontfix
status-seamonkey2.4: --- → fixed
You need to log in before you can comment on or make changes to this bug.