Closed Bug 1106783 Opened 10 years ago Closed 1 year ago

Temporary list artefacts when exiting Mailing List (ML) Properties dialogue via [OK]: Contacts Side Bar shows ML recipients instead of AB contents, and selected ML display in main AB window replaces last list member with an instance of containing ML

Categories

(Thunderbird :: Message Compose Window, defect)

31 Branch
defect

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: thomas8, Unassigned)

References

Details

(Keywords: regression)

+++ This bug was initially created as a clone of Bug #1061648 +++

We did a very good thing in bug 1061648, but in the jungle of the current AB, it's not good enough yet...

Regression from bug 1061648

STR

- create custom TestAB with TestList mailing list having
<foo1@bar.com>,<foo2@bar.com>,<foo3@bar.com>
- in TestAB (NOT in the list!), create more contacts
<foo4@bar.com>,<foo5@bar.com> 
1 select TestList in folder list of main AB view
2 compose new msg, F9 to show contacts side bar, choose TestAB from AB dropdown, and select TestList
3 from contacts side bar, open Properties of TestList (on Windows, do NOT use universal Alt+Enter shortcut for properties because it would be too platform-consistent, too efficient and too safe hence forbidden fruit, waiting for somebody to commit the sweet sin in Bug 998312; until then it's move mouse to where your keyboard focus already is, right-click, navigate context menu, click Properties and make sure that click is precisely positioned or else you might end up deleting the list...)
4 Close Mailing List Properties dialogue using [OK] button
5 Observe content of TestAB list pane in contacts side bar
6 Observe content of TestList list pane in main AB window

Actual result
5 In contacts side bar, TestAB list pane unexpectedly shows only contacts contained in TestList (<foo1@bar.com>,<foo2@bar.com>,<foo3@bar.com>)
6 In main AB window, TestList list pane unexpectedly swaps the last list member <foo3@bar.com> with another functional instance of TestList (wrongly suggesting the TestList contains a copy of itself).
In both cases, selecting something else which forces update of list pane, then going back to TestAB or TestList respectively will clear the wrong stuff

Expected result
5 Contacts sidebar, contents of TestAB should continue to be shown
6 Main AB window, content of TestList should continue to be shown correctly

Stephen, could you look into this? Tia :)
(In reply to Thomas D. from comment #0)
> - create custom TestAB having foo1 with TestList mailing list having

"having foo1" is having too much in that sentence, so just strike those two words
Summary: After confirming Mailing List Properties dialogue with [OK], contacts side bar shows mailing list recipients instead of AB contents, and selected mailing list display in main AB window replaces last list member with an instance of containing mailing list → Temporary list artefacts when exiting Mailing List (ML) Properties dialogue via [OK]: Contacts Side Bar shows ML recipients instead of AB contents, and selected ML display in main AB window replaces last list member with an instance of containing ML
FTR: Seen on Daily 36.0a1 (2014-11-23), using WinXP
Blocks: 434014
I tested the repro steps in comment #0 on build 37.0a1. I had difficulty getting the same results. For example, Actual Result step 6 occurred for me in both the contacts sidebar and the main AB window; I did not observe the behavior described in Actual Result step 5.

I do not believe that this behavior results from the patch for bug 1061648, because when I tested release 31.3.0, which does not contain the patch, I got similar results. Also, when I removed the code for 1061648, the behavior continued to be exhibited. Not sure what's going on here.
Assignee: sschaub → nobody
Status: ASSIGNED → NEW
Severity: normal → minor
Component: Address Book → Message Compose Window
Severity: minor → S4

Unfortunately, it is no longer possible since TB 102 to view or edit a mailing list directly from contacts sidebar (see Bug 1771795 Comment 9, ymmv), which makes this bug obsolete.

Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.