using the test builds hosted by mozilla: If I launch mail for the first time w/ simple mapi installed, the profile manager comes up. I select my profile. Then I get dialog asking me about making mozilla mail the default mail client (for mapi). After dismissing this dialog the mail window then shows up. This is very confusing to me if I were a new user. The dialog box should be coming up on top of the mail window. i.e. the mail window comes up, the user can see that mail has been launched here. Then the modal dialog comes up asking them to make use the default mail client. I would note that the browser dialog for making us the default browser used to work this way. that is, it came up once the first browser window was visible to the user. I'll post a screen shot which will illustrate the strange look.

Nominating for the branch. I suspect Jennifer might have some UE concerns. We shouldn't be putting up this mail dialog without a mail window underneath it. Starting up the app and just seeing this dialog without anything underneath it looks out of place to me. IMHO.

Reassign to Srilatha.

I agree with mscott, that seeing this dialog in the context of the Mail app behind it, will help users understand why they are seeing this dialog.

is windows xp integration on the branch? if so, I think one thing to do while pursuing this fix is to address #102560 I'll work with srilatha on this.

ok, I got a branch build and I can reproduce the bug. investigating a fix...

ok, testing a potential fix. I've put the call to showMailIntegrationDialog() on a instant timer, so it will get executed *after* the window loads. I've also extended the nsIMapiRegistry interface to take a nsIDOMWindow, so that we can pass the parent window (to root the confirm dialog). here comes a patch.

taking from srilatha, since I've got the fix. this fix works if you launch addrbook, compose or 3 pane.

looks good r=srilatha

Comment on attachment 52610 [details] [diff] [review] a fix sr=bienvenu

this patch also fixes a bug with the account wizard and new profiles. when you launch mail for the first time, you'll get the account wizard, and when you close the wizard, then you'll get the integration confirm dialog. I can't land this on the trunk because the mapi code is not on the trunk yet. once this land on the branch, I'll re-assign to srilatha, so she can land it on the trunk.

This sounds like an interesitng bug. Can we talk about it, in the PDT tomorrow?

pls check this into the 094 branch - PDT+

fixed on the branch, marking fixed since this is branch only. srilatha, please add this to your list of patches to bring over when mapi lands on the trunk.

verified on 2001-10-15-05-0.9.4

The patch (attachment id=52610) needs to be landed on the trunk. Seth can u please sr this for the trunk along with MAPI trunk landing ? thanks, - Rajiv.

I'm seeing something similar to this in 0.9.8 (filed bug 127762). It would be nice if somebody could take a look to see if it's a regression.