Closed
Bug 814770
Opened 12 years ago
Closed 12 years ago
Select Addresses blank if opened from Compose window with To/Cc/Bcc filled in
Categories
(SeaMonkey :: MailNews: Address Book & Contacts, defect)
SeaMonkey
MailNews: Address Book & Contacts
Tracking
(seamonkey2.14 fixed, seamonkey2.15 fixed, seamonkey2.16 fixed)
RESOLVED
FIXED
seamonkey2.17
People
(Reporter: InvisibleSmiley, Assigned: neil)
References
Details
(Keywords: regression, relnote)
Attachments
(1 file)
782 bytes,
patch
|
InvisibleSmiley
:
review+
InvisibleSmiley
:
approval-comm-aurora+
InvisibleSmiley
:
approval-comm-beta+
iannbugzilla
:
approval-comm-release+
|
Details | Diff | Splinter Review |
STR: 0. Ensure you have a mail account and contact(s) in the Personal Address Book 1. Open a Compose window (Ctrl+M) 2. Fill in at least one To, Cc or Bcc line 3. Click the Address icon or select Options/Select Addresses Actual result: The list on the left (with headers Name, Email, Screen Name) is empty Expected result: The list on the left shows your contacts. You will get this result if you have no (filled in) To, Cc or Bcc line in the Compose window. Last known good: SM 2.13.2 First bad: 2.14b1 Nothing in the Error Console.
Reporter | ||
Comment 1•12 years ago
|
||
It breaks here http://mxr.mozilla.org/comm-central/source/suite/mailnews/addrbook/abSelectAddressesDialog.js#121 because gHeaderParser is undefined since bug 774069 removed it from abCommon.js.
Depends on: 774069
Reporter | ||
Updated•12 years ago
|
Component: MailNews: Composition → MailNews: Address Book & Contacts
Assignee | ||
Comment 2•12 years ago
|
||
Reporter | ||
Updated•12 years ago
|
Attachment #684860 -
Flags: review?(jh) → review+
Assignee | ||
Comment 3•12 years ago
|
||
Pushed comm-central changeset 90c30f23e058.
Assignee | ||
Comment 4•12 years ago
|
||
Comment on attachment 684860 [details] [diff] [review] Proposed patch [Approval Request Comment] Regression caused by (bug #): 774069 User impact if declined: Can't edit existing addreses using dialog Testing completed (on m-c, etc.): Risk to taking this patch (and alternatives if risky): Low String changes made by this patch: None
Attachment #684860 -
Flags: approval-comm-beta?
Attachment #684860 -
Flags: approval-comm-aurora?
Reporter | ||
Updated•12 years ago
|
Attachment #684860 -
Flags: approval-comm-beta?
Attachment #684860 -
Flags: approval-comm-beta+
Attachment #684860 -
Flags: approval-comm-aurora?
Attachment #684860 -
Flags: approval-comm-aurora+
Reporter | ||
Updated•12 years ago
|
status-seamonkey2.17:
affected → ---
Target Milestone: --- → seamonkey2.17
Assignee | ||
Comment 5•12 years ago
|
||
https://hg.mozilla.org/releases/comm-aurora/rev/c181ca27c7fb https://hg.mozilla.org/releases/comm-beta/rev/a47244f34da9
Comment on attachment 684860 [details] [diff] [review] Proposed patch [Triage Comment]
Attachment #684860 -
Flags: approval-comm-release+
Assignee | ||
Comment 8•12 years ago
|
||
Comment on attachment 684860 [details] [diff] [review] Proposed patch https://hg.mozilla.org/releases/comm-release/rev/a44252ba0f5d
Assignee | ||
Updated•12 years ago
|
Reporter | ||
Updated•12 years ago
|
tracking-seamonkey2.14:
? → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•