Problems in folder pane after a new IMAP account is created

VERIFIED FIXED in mozilla0.9.9

Status

SeaMonkey
MailNews: Message Display
--
major
VERIFIED FIXED
17 years ago
14 years ago

People

(Reporter: racham, Assigned: Chris Waterson)

Tracking

({regression})

Trunk
mozilla0.9.9
x86
Windows 2000
regression
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(8 attachments)

(Reporter)

Description

17 years ago
Here are the problems I have noticed with steps to reproduce:

1. Create a new profile
2. Launch mailnews
3. Create a new mail account
4. This is your default account. So, in folder pane, the account should be 
twisty opened and Inbox should be selected. I see it twisty opened, but there is 
 no Inbox. Password dialog has not come up. That means Inbox (invisible at this 
stage) is not selected.
5. Focus in folderpane is lost. But on the right hand side, you see threadpane 
and message pane which means the focus in folder should have been on one of the 
folders, Inbox in this case. That is not happening as mentioned earlier.

Problem #2 

6. Now click on the account in folder pane. Notice the Account Central page on 
the right hand side.
7. Click on 'Read Messages'. 
8. It prompts you for the passsword.
9. Enter the password and notice your imap folders getting shown up except the 
INBOX folder.
10. The focus in the folder pane is still on the account as Inbox is not found.

I know Inbox is there and is just not getting displayed.

Here is what brings life to Inbox.

-> Click on the twisty of the account to close it once and click on it to expand 
and notice Inbox folder show up.

So, manual clicking on the twisty is doing the right thing. But programmatic 
twisty isn't doing the samething possibly causing all these problems.

Will post some screen shots.
(Reporter)

Comment 1

17 years ago
Created attachment 46967 [details]
screen shot #1 : (default) Account twisty is opened, but no Inbox seen, immediately after account creation
(Reporter)

Comment 2

17 years ago
Created attachment 46968 [details]
screen shot #2 : Account is twisty opened when clicked on 'Read Messages' in Account Central. After login, all folders are seen except Inbox
(Reporter)

Comment 3

17 years ago
Another related problem I noticed :

1. Create a one more mail account.
2. Notice that this account's twisty is shown open in the folder pane. We do 
that only for default account.

Note : Check new mail pref is set corerctly for all these accounts (checked for 
default account, unchecked for additional account).

(Reporter)

Comment 4

17 years ago
Created attachment 46970 [details]
screen shot #3 : twisty for additional account is open. It shouldn't be.

Updated

17 years ago
QA Contact: esther → nbaca

Updated

17 years ago
Keywords: nsbranch
per PDT triage, adding sspitzer to this bug. mscott or seth - can you please 
make the call on this bug? Thanks. 

Comment 6

17 years ago
hey seth, this seems kinda funky...plussing for now. What's your take on it? 
Keywords: nsbranch → nsbranch+
Target Milestone: --- → mozilla0.9.5
yes, I see this and some similar problems when creating a new news account and 
subscribe to the first newsgroup.
Status: NEW → ASSIGNED
I think the problem (in the case of imap) is that we assert() that the inbox 
got added during the assert() that the server got added.

the nsXULTemplateBuilder "ignores re-entrant updates while we're building 
content".

nsXULTemplateBuilder::OnAssert(nsXULTemplateBuilder * const 0x04145c0c, 
nsIRDFDataSource * 0x04145270, nsIRDFResource * 0x0598de30, nsIRDFResource * 
0x015c9780, nsIRDFNode * 0x0599e790) line 553
CompositeDataSourceImpl::OnAssert(CompositeDataSourceImpl * const 0x04145274, 
nsIRDFDataSource * 0x04dbdc50, nsIRDFResource * 0x0598de30, nsIRDFResource * 
0x015c9780, nsIRDFNode * 0x0599e790) line 1543
nsMsgRDFDataSource::assertEnumFunc(nsISupports * 0x04145274, void * 0x00129574) 
line 400
nsSupportsArray::EnumerateForwards(nsSupportsArray * const 0x04dbdbe0, int 
(nsISupports *, void *)* 0x02df4250 nsMsgRDFDataSource::assertEnumFunc
(nsISupports *, void *), void * 0x00129574) line 669 + 20 bytes
nsMsgRDFDataSource::NotifyObservers(nsIRDFResource * 0x0598de30, nsIRDFResource 
* 0x015c9780, nsIRDFNode * 0x0599e790, int 1, int 0) line 384
nsMsgFolderDataSource::OnItemAddedOrRemoved(nsISupports * 0x0598de30, 
nsISupports * 0x0599e790, const char * 0x0592fb4c, int 1) line 820
nsMsgFolderDataSource::OnItemAdded(nsMsgFolderDataSource * const 0x04dbdc80, 
nsISupports * 0x0598de30, nsISupports * 0x0599e790, const char * 0x0592fb4c) 
line 786
nsMsgMailSession::OnItemAdded(nsMsgMailSession * const 0x04186cf4, nsISupports 
* 0x0598de30, nsISupports * 0x0599e790, const char * 0x0592fb4c) line 251
nsMsgFolder::NotifyItemAdded(nsMsgFolder * const 0x0598de4c, nsISupports * 
0x0598de30, nsISupports * 0x0599e790, const char * 0x0592fb4c) line 2399
nsImapMailFolder::CreateClientSubfolderInfo(nsImapMailFolder * const 
0x0598df08, const char * 0x0592fa34, unsigned short 94, int 0) line 832
nsImapMailFolder::GetSubFolders(nsImapMailFolder * const 0x0598de4c, 
nsIEnumerator * * 0x00129be4) line 520
nsMsgFolderDataSource::createFolderChildNode(nsIMsgFolder * 0x0598de4c, 
nsIRDFNode * * 0x00129d5c) line 1746 + 36 bytes
nsMsgFolderDataSource::createFolderNode(nsIMsgFolder * 0x0598de4c, 
nsIRDFResource * 0x015c9780, nsIRDFNode * * 0x00129d5c) line 1013 + 16 bytes
nsMsgFolderDataSource::GetTarget(nsMsgFolderDataSource * const 0x04dbdc50, 
nsIRDFResource * 0x0598de30, nsIRDFResource * 0x015c9780, int 1, nsIRDFNode * * 
0x00129d5c) line 341 + 25 bytes
CompositeDataSourceImpl::GetTarget(CompositeDataSourceImpl * const 0x04145270, 
nsIRDFResource * 0x0598de30, nsIRDFResource * 0x015c9780, int 1, nsIRDFNode * * 
0x00129d5c) line 816 + 28 bytes
nsRDFConInstanceTestNode::FilterInstantiations(InstantiationSet & {...}, void * 
0x00129f18) line 160 + 95 bytes
TestNode::Propogate(const InstantiationSet & {...}, void * 0x00129f18) line 
1030 + 19 bytes
TestNode::Propogate(const InstantiationSet & {...}, void * 0x00129f18) line 1040
TestNode::Propogate(const InstantiationSet & {...}, void * 0x00129f18) line 1040
nsXULTemplateBuilder::Propogate(nsIRDFResource * 0x04156f60, nsIRDFResource * 
0x015c9780, nsIRDFNode * 0x0598de30, nsClusterKeySet & {...}) line 496 + 19 
bytes
nsXULTemplateBuilder::OnAssert(nsXULTemplateBuilder * const 0x04145c0c, 
nsIRDFDataSource * 0x04145270, nsIRDFResource * 0x04156f60, nsIRDFResource * 
0x015c9780, nsIRDFNode * 0x0598de30) line 566 + 27 bytes
CompositeDataSourceImpl::OnAssert(CompositeDataSourceImpl * const 0x04145274, 
nsIRDFDataSource * 0x04dbc3f0, nsIRDFResource * 0x04156f60, nsIRDFResource * 
0x015c9780, nsIRDFNode * 0x0598de30) line 1543
nsMsgRDFDataSource::assertEnumFunc(nsISupports * 0x04145274, void * 0x00129ff8) 
line 400
nsSupportsArray::EnumerateForwards(nsSupportsArray * const 0x04dbc380, int 
(nsISupports *, void *)* 0x02df4250 nsMsgRDFDataSource::assertEnumFunc
(nsISupports *, void *), void * 0x00129ff8) line 669 + 20 bytes
nsMsgRDFDataSource::NotifyObservers(nsIRDFResource * 0x04156f60, nsIRDFResource 
* 0x015c9780, nsIRDFNode * 0x0598de30, int 1, int 0) line 384
nsMsgAccountManagerDataSource::OnServerLoaded(nsMsgAccountManagerDataSource * 
const 0x04dbc424, nsIMsgIncomingServer * 0x0598d5d0) line 994
nsMsgAccountManager::NotifyServerLoaded(nsMsgAccountManager * const 0x031db560, 
nsIMsgIncomingServer * 0x0598d5d0) line 1572
nsMsgAccount::SetIncomingServer(nsMsgAccount * const 0x0598d780, 
nsIMsgIncomingServer * 0x0598d5d0) line 193
XPTC_InvokeByIndex(nsISupports * 0x0598d780, unsigned int 7, unsigned int 1, 
nsXPTCVariant * 0x0012a29c) line 139
XPCWrappedNative::CallMethod(XPCCallContext & {...}, XPCWrappedNative::CallMode 
CALL_SETTER) line 1952 + 42 bytes
XPCWrappedNative::SetAttribute(XPCCallContext & {...}) line 1790 + 14 bytes
XPC_WN_GetterSetter(JSContext * 0x05666e70, JSObject * 0x0451d798, unsigned int 
1, long * 0x045ae0b0, long * 0x0012a4e8) line 1278 + 12 bytes
js_Invoke(JSContext * 0x05666e70, unsigned int 1, unsigned int 2) line 807 + 23 
bytes
js_InternalInvoke(JSContext * 0x05666e70, JSObject * 0x0451d798, long 72472496, 
unsigned int 0, unsigned int 1, long * 0x0012b2cc, long * 0x0012b2cc) line 899 
+ 20 bytes
js_SetProperty(JSContext * 0x05666e70, JSObject * 0x0451d798, long 52963120, 
long * 0x0012b2cc) line 2674 + 47 bytes
js_Interpret(JSContext * 0x05666e70, long * 0x0012b484) line 2564 + 1939 bytes
js_Invoke(JSContext * 0x05666e70, unsigned int 1, unsigned int 2) line 824 + 13 
bytes
js_InternalInvoke(JSContext * 0x05666e70, JSObject * 0x04523008, long 72466504, 
unsigned int 0, unsigned int 1, long * 0x0012b664, long * 0x0012b5ac) line 899 
+ 20 bytes
JS_CallFunctionValue(JSContext * 0x05666e70, JSObject * 0x04523008, long 
72466504, unsigned int 1, long * 0x0012b664, long * 0x0012b5ac) line 3362 + 31 
bytes
nsJSContext::CallEventHandler(nsJSContext * const 0x05660280, void * 
0x04523008, void * 0x0451c048, unsigned int 1, void * 0x0012b664, int * 
0x0012b660, int 0) line 960 + 33 bytes
nsJSEventListener::HandleEvent(nsJSEventListener * const 0x015daad0, 
nsIDOMEvent * 0x059b3dc4) line 139 + 74 bytes
nsEventListenerManager::HandleEventSubType(nsListenerStruct * 0x015daa30, 
nsIDOMEvent * 0x059b3dc4, nsIDOMEventTarget * 0x015dd468, unsigned int 8, 
unsigned int 7) line 1197 + 20 bytes
nsEventListenerManager::HandleEvent(nsEventListenerManager * const 0x015dab20, 
nsIPresContext * 0x0595f5f0, nsEvent * 0x0012c174, nsIDOMEvent * * 0x0012c038, 
nsIDOMEventTarget * 0x015dd468, unsigned int 7, nsEventStatus * 0x0012c1bc) 
line 2187 + 36 bytes
nsXULElement::HandleDOMEvent(nsXULElement * const 0x015dd460, nsIPresContext * 
0x0595f5f0, nsEvent * 0x0012c174, nsIDOMEvent * * 0x0012c038, unsigned int 1, 
nsEventStatus * 0x0012c1bc) line 3711
PresShell::HandleDOMEventWithTarget(PresShell * const 0x059511c0, nsIContent * 
0x015dd460, nsEvent * 0x0012c174, nsEventStatus * 0x0012c1bc) line 5755 + 39 
bytes
nsButtonBoxFrame::MouseClicked(nsIPresContext * 0x0595f5f0, nsGUIEvent * 
0x0012c370) line 181
nsButtonBoxFrame::HandleEvent(nsButtonBoxFrame * const 0x04536f6c, 
nsIPresContext * 0x0595f5f0, nsGUIEvent * 0x0012c370, nsEventStatus * 
0x0012c67c) line 128
PresShell::HandleEventInternal(nsEvent * 0x0012c370, nsIView * 0x00000000, 
unsigned int 1, nsEventStatus * 0x0012c67c) line 5723 + 41 bytes
PresShell::HandleEventWithTarget(PresShell * const 0x059511c0, nsEvent * 
0x0012c370, nsIFrame * 0x04536f6c, nsIContent * 0x015dd460, unsigned int 1, 
nsEventStatus * 0x0012c67c) line 5679 + 22 bytes
nsEventStateManager::CheckForAndDispatchClick(nsEventStateManager * const 
0x02790140, nsIPresContext * 0x0595f5f0, nsMouseEvent * 0x0012c788, 
nsEventStatus * 0x0012c67c) line 2486 + 61 bytes
nsEventStateManager::PostHandleEvent(nsEventStateManager * const 0x02790148, 
nsIPresContext * 0x0595f5f0, nsEvent * 0x0012c788, nsIFrame * 0x04536f6c, 
nsEventStatus * 0x0012c67c, nsIView * 0x0595cd10) line 1571 + 28 bytes
PresShell::HandleEventInternal(nsEvent * 0x0012c788, nsIView * 0x0595cd10, 
unsigned int 1, nsEventStatus * 0x0012c67c) line 5728 + 43 bytes
PresShell::HandleEvent(PresShell * const 0x059511c4, nsIView * 0x0595cd10, 
nsGUIEvent * 0x0012c788, nsEventStatus * 0x0012c67c, int 1, int & 1) line 5633 
+ 25 bytes
nsView::HandleEvent(nsView * const 0x0595cd10, nsGUIEvent * 0x0012c788, 
unsigned int 28, nsEventStatus * 0x0012c67c, int 1, int & 1) line 377
nsViewManager::DispatchEvent(nsViewManager * const 0x0595ceb0, nsGUIEvent * 
0x0012c788, nsEventStatus * 0x0012c67c) line 2062
HandleEvent(nsGUIEvent * 0x0012c788) line 68
nsWindow::DispatchEvent(nsWindow * const 0x0595cbd4, nsGUIEvent * 0x0012c788, 
nsEventStatus & nsEventStatus_eIgnore) line 728 + 10 bytes
nsWindow::DispatchWindowEvent(nsGUIEvent * 0x0012c788) line 749
nsWindow::DispatchMouseEvent(unsigned int 301, nsPoint * 0x00000000) line 4262 
+ 21 bytes
ChildWindow::DispatchMouseEvent(unsigned int 301, nsPoint * 0x00000000) line 
4514
nsWindow::ProcessMessage(unsigned int 514, unsigned int 0, long 22872518, long 
* 0x0012cbb4) line 3221 + 24 bytes
nsWindow::WindowProc(HWND__ * 0x003602c0, unsigned int 514, unsigned int 0, 
long 22872518) line 996 + 27 bytes
USER32! 77e13eb0()
USER32! 77e1401a()
USER32! 77e192da()
nsXULWindow::ShowModal(nsXULWindow * const 0x056631c0) line 277
nsWebShellWindow::ShowModal(nsWebShellWindow * const 0x056631c0) line 1041
nsContentTreeOwner::ShowAsModal(nsContentTreeOwner * const 0x056688bc) line 415
nsWindowWatcher::OpenWindowJS(nsWindowWatcher * const 0x0156b9c4, nsIDOMWindow 
* 0x0278a374, const char * 0x05602920, const char * 0x05602a90, const char * 
0x056028d0, int 1, unsigned int 0, long * 0x0453ce2c, nsIDOMWindow * * 
0x0012d2fc) line 701
GlobalWindowImpl::OpenInternal(GlobalWindowImpl * const 0x0278a370, const 
nsAString & {...}, const nsAString & {...}, const nsAString & {...}, int 1, 
long * 0x0453ce20, unsigned int 3, nsISupports * 0x00000000, nsIDOMWindow * * 
0x0012d6cc) line 3161 + 129 bytes
GlobalWindowImpl::OpenDialog(GlobalWindowImpl * const 0x0278a378, nsIDOMWindow 
* * 0x0012d6cc) line 2356 + 59 bytes
XPTC_InvokeByIndex(nsISupports * 0x0278a378, unsigned int 14, unsigned int 1, 
nsXPTCVariant * 0x0012d6cc) line 139
XPCWrappedNative::CallMethod(XPCCallContext & {...}, XPCWrappedNative::CallMode 
CALL_METHOD) line 1952 + 42 bytes
XPC_WN_CallMethod(JSContext * 0x04b6dcd0, JSObject * 0x01043878, unsigned int 
3, long * 0x0453ce20, long * 0x0012d904) line 1254 + 14 bytes
js_Invoke(JSContext * 0x04b6dcd0, unsigned int 3, unsigned int 0) line 807 + 23 
bytes
js_Interpret(JSContext * 0x04b6dcd0, long * 0x0012e6a8) line 2719 + 15 bytes
js_Invoke(JSContext * 0x04b6dcd0, unsigned int 1, unsigned int 2) line 824 + 13 
bytes
js_InternalInvoke(JSContext * 0x04b6dcd0, JSObject * 0x0451de88, long 72474264, 
unsigned int 0, unsigned int 1, long * 0x0012e888, long * 0x0012e7d0) line 899 
+ 20 bytes
JS_CallFunctionValue(JSContext * 0x04b6dcd0, JSObject * 0x0451de88, long 
72474264, unsigned int 1, long * 0x0012e888, long * 0x0012e7d0) line 3362 + 31 
bytes
nsJSContext::CallEventHandler(nsJSContext * const 0x04b6c030, void * 
0x0451de88, void * 0x0451de98, unsigned int 1, void * 0x0012e888, int * 
0x0012e884, int 0) line 960 + 33 bytes
nsJSEventListener::HandleEvent(nsJSEventListener * const 0x053ac940, 
nsIDOMEvent * 0x0565c604) line 139 + 74 bytes
nsEventListenerManager::HandleEventSubType(nsListenerStruct * 0x053ac8a0, 
nsIDOMEvent * 0x0565c604, nsIDOMEventTarget * 0x053ac9f8, unsigned int 4, 
unsigned int 7) line 1197 + 20 bytes
nsEventListenerManager::HandleEvent(nsEventListenerManager * const 0x053ac990, 
nsIPresContext * 0x0538b8f0, nsEvent * 0x0012f350, nsIDOMEvent * * 0x0012f25c, 
nsIDOMEventTarget * 0x053ac9f8, unsigned int 7, nsEventStatus * 0x0012f65c) 
line 1365 + 36 bytes
nsXULElement::HandleDOMEvent(nsXULElement * const 0x053ac9f0, nsIPresContext * 
0x0538b8f0, nsEvent * 0x0012f350, nsIDOMEvent * * 0x0012f25c, unsigned int 1, 
nsEventStatus * 0x0012f65c) line 3711
PresShell::HandleEventInternal(nsEvent * 0x0012f350, nsIView * 0x00000000, 
unsigned int 1, nsEventStatus * 0x0012f65c) line 5708 + 47 bytes
PresShell::HandleEventWithTarget(PresShell * const 0x0538cb40, nsEvent * 
0x0012f350, nsIFrame * 0x044d09c0, nsIContent * 0x053ac9f0, unsigned int 1, 
nsEventStatus * 0x0012f65c) line 5679 + 22 bytes
nsEventStateManager::CheckForAndDispatchClick(nsEventStateManager * const 
0x053c3c20, nsIPresContext * 0x0538b8f0, nsMouseEvent * 0x0012f768, 
nsEventStatus * 0x0012f65c) line 2486 + 61 bytes
nsEventStateManager::PostHandleEvent(nsEventStateManager * const 0x053c3c28, 
nsIPresContext * 0x0538b8f0, nsEvent * 0x0012f768, nsIFrame * 0x044d09c0, 
nsEventStatus * 0x0012f65c, nsIView * 0x0538b090) line 1571 + 28 bytes
PresShell::HandleEventInternal(nsEvent * 0x0012f768, nsIView * 0x0538b090, 
unsigned int 1, nsEventStatus * 0x0012f65c) line 5728 + 43 bytes
PresShell::HandleEvent(PresShell * const 0x0538cb44, nsIView * 0x0538b090, 
nsGUIEvent * 0x0012f768, nsEventStatus * 0x0012f65c, int 1, int & 1) line 5633 
+ 25 bytes
nsView::HandleEvent(nsView * const 0x0538b090, nsGUIEvent * 0x0012f768, 
unsigned int 28, nsEventStatus * 0x0012f65c, int 1, int & 1) line 377
nsViewManager::DispatchEvent(nsViewManager * const 0x0538d250, nsGUIEvent * 
0x0012f768, nsEventStatus * 0x0012f65c) line 2062
HandleEvent(nsGUIEvent * 0x0012f768) line 68
nsWindow::DispatchEvent(nsWindow * const 0x05389044, nsGUIEvent * 0x0012f768, 
nsEventStatus & nsEventStatus_eIgnore) line 728 + 10 bytes
nsWindow::DispatchWindowEvent(nsGUIEvent * 0x0012f768) line 749
nsWindow::DispatchMouseEvent(unsigned int 301, nsPoint * 0x00000000) line 4262 
+ 21 bytes
ChildWindow::DispatchMouseEvent(unsigned int 301, nsPoint * 0x00000000) line 
4514
nsWindow::ProcessMessage(unsigned int 514, unsigned int 0, long 7405661, long * 
0x0012fb94) line 3221 + 24 bytes
nsWindow::WindowProc(HWND__ * 0x00340324, unsigned int 514, unsigned int 0, 
long 7405661) line 996 + 27 bytes
USER32! 77e13eb0()
USER32! 77e1401a()
USER32! 77e192da()
nsAppShellService::Run(nsAppShellService * const 0x01569030) line 442
main1(int 4, char * * 0x00485ce0, nsISupports * 0x00000000) line 1278 + 32 bytes
main(int 4, char * * 0x00485ce0) line 1606 + 37 bytes
mainCRTStartup() line 338 + 17 bytes
KERNEL32! 77e87903()
two (possibly) related bugs:

1) new pop accounts get created and they are closed.  (they should be open)
2) new news accounts get created, and when you first add a newsgroup, there is 
no twisty.
I've got a hack, but I'm not happy with it.

it works around the problem by making the account closed by default on creation.

but, both imap and pop accounts should be open on creation.

I'll continue to work on this.
minusing, but I'll continue to work on this.  if I keep getting stumped, I'll 
hand off to waterson.
Keywords: nsbranch+ → nsbranch-
> 1) new pop accounts get created and they are closed.  (they should be open)

I'll go log a bug on this.

> 2) new news accounts get created, and when you first add a newsgroup, there 
is no twisty.

waterson fix for #97859 will fix that.  (there's already a bug for this, I've 
marked it a dup of #97859)
Summary: Problems in folder pane after a new account is created → Problems in folder pane after a new IMAP account is created
new pop account bug:  #102225
(Assignee)

Comment 15

17 years ago
Stealing, almost certainly a bug in rdfliner.
Assignee: sspitzer → waterson
Status: ASSIGNED → NEW
(Assignee)

Comment 16

17 years ago
WRT sspitzer's comments from 2001-09-20 23:15: yes, the template builder isn't
re-entrant, but so long as GetTarget() returns something reasonable, everything
should come out in the wash.
Status: NEW → ASSIGNED
(Assignee)

Comment 17

17 years ago
Created attachment 51815 [details] [diff] [review]
check new rows to see if they need to be opened
(Assignee)

Comment 18

17 years ago
Well, sho'nuff, it didn't really have anything to do with re-entrancy. I wasn't
checking newly added rows to see if they were being added as ``open''. The fix
is to check after adding the row, and if so, do the necessary dance to add the
new row's children to the outliner too.
(Assignee)

Updated

17 years ago
Keywords: patch
excellent!  I tried it out and it fixes the problem.  (I tested on the branch).

Updated

17 years ago
Blocks: 99508
Keywords: nsbranch- → nsbranch+, regression

Comment 20

17 years ago
You rock Chris. Thanks so much for looking into this for us. Making this a
nsBranch+ bug. Assuming the reviews happen by 3:30pm tomorrow, I'll take it to
PDT and see what they say. I think the odds are very good since this fixes a UI
regression in the 3-pane and marketing is very interested in taking those kinds
of fixes. Thanks again.

Comment 21

17 years ago
Nice work, r=varga

Comment 22

17 years ago
sr=hyattbot
(Assignee)

Comment 23

17 years ago
Fix checked in on the trunk.
Whiteboard: fixed on trunk

Comment 24

17 years ago
Looks like something we'd like to take. Marking [PDT]. Pls update the Patch
Status, when you get a chance.
Whiteboard: fixed on trunk → fixed on trunk [PDT]
(Assignee)

Comment 25

17 years ago
Comment on attachment 51815 [details] [diff] [review]
check new rows to see if they need to be opened

r=varga@utcru.sk, sr=hyatt
Attachment #51815 - Flags: superreview+
Attachment #51815 - Flags: review+

Comment 26

17 years ago
Trunk build 2001-10-04: WinMe

I would like to see this fix in the Branch build since it is an improvement.

Results so far:
1. This portion is fixed, auto login and no double Inbox: New profile, add 1st
IMAP account and the account is expanded so I see the Inbox, a password dialog
appears (known problem with it being cutoff), enter password and messages are
retrieved as expected.
  ------------------------------------------------------------------
This part is not fixed but may require its own bug:

2. Add second IMAP account and the widget is expanded but no Inbox appears. 
Workaround: I have to close/expand the widget to see the Inbox, select the Inbox
for the password dialog to appear (same results using the 10/4 branch build).

6.1RTM behavior: When the second account is added the account widget is
collapsed. To login I have to expand the widget and select the Inbox. 

After discussing this with jglick maybe a good compromise would be to expand the
second account but leave focus on the original account.

Comment 27

17 years ago
pls check this into the branch - PDT+
Whiteboard: fixed on trunk [PDT] → fixed on trunk [PDT+]

Comment 28

17 years ago
Trunk build 2001-10-04: WinMe
I have a concern regarding adding the second account. After the second account
is added, I'm seeing double Inboxes under the first account. Please try this
scenario:

- New profile
- Add the 1st IMAP account, login (qatest22)
- Add a 2nd IMAP account (qatest33)

Actual Results: Look at the first account in the folder pane. Notice it now has
2 Inboxes. Click on the second Inbox and a password dialog for the second
account appears (qatest33). Scroll down the folder pane and notice that the
second account does not have an Inbox directly under its account level.

Comment 29

17 years ago
Created attachment 52150 [details]
After adding 2nd acct, two Inboxes

Comment 30

17 years ago
Created attachment 52151 [details]
Click on 2nd Inbox, password dialog for 2nd account appears

Comment 31

17 years ago
Created attachment 52152 [details]
After login to 2nd acct, scroll to second account and no Inbox

Comment 32

17 years ago
Branch build 2001-10-04: WinMe
I don't see double Inboxes for the first account. When the second account is
added I still don't see double Inboxes. The problem with both accounts is that
the account widget is expanded yet no folders appear, I have to close/expand the
account widget to see the Inbox, and have to select the Inbox to get a password
dialog.
(Assignee)

Comment 33

17 years ago
Fixed on MOZILLA_0_9_4_BRANCH.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 34

17 years ago
Branch build 2001-10-05-05: WinMe
Branch build 2001-10-05-03: Mac 9.1
Branch build 2001-10-05-04: Linux RH 7.1
- 1st account: only 1 Inbox appears
- 2nd account: now the 1st account appears to have 2 Inboxes! I have the same
concerns regarding this issue. Is this going to be fixed? After an exit/restart
it's ok but it's terribley confusing.

Reopening due to this concern.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Comment 35

17 years ago
Summary of additional testing: 

- When the second account added is an IMAP account, added via Acct Wizard (i.e.
nsmail-2, webmail, aol) then the double Inbox appears under the first account. 
- If the second account is an Activated webmail account then it's ok.
- If the second account is POP or News then it's ok

     ---------------------------------------------------------------------

Scenarios tried in a new profile:
- POP/POP - ok
- POP/IMAP - double inbox
- POP/Webmail (using Acct Wiz) - double inbox
- IMAP/POP - ok
- IMAP/IMAP - double inbox
- IMAP/AOL (using Acct Wiz) - double inbox
- Configured for a POP account, exit/restart the appliation, added IMAP account
and the double inbox still appears.
- POP/News - ok

Activation scenarios:
- Migrated POP acct, Activated WebMail account - ok 
- New Profile POP, exit/restart the application, Activate WebMail account - ok


Workaround: When the double Inboxes appear then close/expand the account widget
(or close/open Mail). 
(Assignee)

Comment 36

17 years ago
Okay, although I think that the double-inbox bug should be fixed if we can
figure out something for bug 100952.
Status: REOPENED → ASSIGNED
Depends on: 100952
Target Milestone: mozilla0.9.5 → mozilla0.9.6

Comment 37

17 years ago
I forgot to mention that we took this out of the 094 branch since it ended up
making things worse (with creating a 2nd account). Removing the branch+ since
this will end up getting fixed on the trunk only. 

Leaving open for work on the trunk.
Keywords: nsbranch+
Whiteboard: fixed on trunk [PDT+]

Comment 38

17 years ago
Branch build 2001-10-11: WinMe, Linux RH 7.1, Mac 9.1
The problem as originally described is now occuring. This is good since the
"fix" made it worse when a second account was added. Will wait for the real fix
on the trunk.
cc jar, who just ran into this on some last minute 0.9.4 bits.
(Assignee)

Updated

17 years ago
Target Milestone: mozilla0.9.6 → mozilla0.9.7

Comment 40

17 years ago
I also just ran into it, if you delete INBOX.msf and start-up again. 
(Assignee)

Updated

17 years ago
Target Milestone: mozilla0.9.7 → mozilla0.9.8
(Assignee)

Updated

17 years ago
Target Milestone: mozilla0.9.8 → mozilla0.9.9
(Assignee)

Updated

17 years ago
Depends on: 119504
(Assignee)

Comment 41

17 years ago
Fixed with checkin for bug 119504.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago17 years ago
Resolution: --- → FIXED

Updated

17 years ago
Severity: normal → major

Comment 42

17 years ago
Trunk build 2002-03-01: WinMe, Linux RH 7.1, Mac 9.1
Verified Fixed. The double inbox does not appear.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.