Closed
Bug 138048
Opened 23 years ago
Closed 23 years ago
double click on card in addressbook side bar, doesn't bring up compose window.
Categories
(SeaMonkey :: MailNews: Address Book & Contacts, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.0.1
People
(Reporter: sspitzer, Assigned: sspitzer)
References
Details
(Keywords: regression, Whiteboard: [fixed on trunk and branch] [adt2 rtm])
Attachments
(1 file)
1.26 KB,
patch
|
racham
:
review+
Bienvenu
:
superreview+
jesup
:
approval+
|
Details | Diff | Splinter Review |
double click on card in addressbook side bar, doesn't bring up compose window.
you get a js error:
JavaScript error:
chrome://messenger/content/addressbook/abSelectAddressesDialog.js line 245: body
has no properties
a regression caused by the fix for #121160
fix in hand. the fix is to re-order how we include the .js files, as we want
the addressbook-panels.js last, so that we use it's AbResultsPaneDoubleClick()
instead of the one in abSelectAddressesDialog.js
Assignee | ||
Comment 1•23 years ago
|
||
Assignee | ||
Comment 2•23 years ago
|
||
accepting.
Comment 3•23 years ago
|
||
Comment on attachment 79683 [details] [diff] [review]
fix
sr=bienvenu
Attachment #79683 -
Flags: superreview+
Comment on attachment 79683 [details] [diff] [review]
fix
r=bhuvan
Attachment #79683 -
Flags: review+
Assignee | ||
Updated•23 years ago
|
Whiteboard: [fix in hand, have r=, sr=, will land on trunk today]
Assignee | ||
Comment 5•23 years ago
|
||
fixed on trunk.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Keywords: adt1.0.0
Resolution: --- → FIXED
Whiteboard: [fix in hand, have r=, sr=, will land on trunk today] → [fixed on trunk]
Comment 6•23 years ago
|
||
Trunk build 2002-04-19-03: WinMe
Trunk build 2002-04-18-22: Linux RH 7.1
Trunk build 2002-04-19-03: Mac 10.1.3
Verified Fixed on the trunk. Double click on card in Sidebar Address Book and
the card's email address is prefilled in the Compose address area as expected.
Status: RESOLVED → VERIFIED
Comment 7•23 years ago
|
||
nsbeta1+/adt3. Yes, this is a regression, but not very severe. We can up the
Impact marking, if we think this hits a lot of users, and is determined to be a
very visible regression, or gross loss of functionality.
Comment 8•23 years ago
|
||
Adding adt1.0.0- and raising impact. Let's reconsider this for rtm.
Updated•23 years ago
|
Comment 9•23 years ago
|
||
adt1.0.1+ (on ADT's behalf) for checkin to the 1.0 branch, pending Driver's
approval.
Blocks: 143047
Comment 11•23 years ago
|
||
Comment on attachment 79683 [details] [diff] [review]
fix
please check into the 1.0.1 branch ASAP. once landed remove the
mozilla1.0.1+ keyword and add the fixed1.0.1 keyword
Attachment #79683 -
Flags: approval+
Updated•23 years ago
|
Keywords: mozilla1.0.1 → mozilla1.0.1+
Assignee | ||
Comment 12•23 years ago
|
||
fixed on trunk and branch.
Keywords: mozilla1.0.1+ → fixed1.0.1
Whiteboard: [fixed on trunk] [adt2 rtm] → [fixed on trunk and branch] [adt2 rtm]
Comment 13•23 years ago
|
||
Branch build 2002-06-11: WinMe
verified1.0.1
Keywords: fixed1.0.1 → verified1.0.1
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•