Open Bug 117130 Opened 23 years ago Updated 16 years ago

Mail Startup: delay sidebar loading until after the password dialog is dismissed.

Categories

(SeaMonkey :: MailNews: Message Display, defect)

All
Windows NT
defect
Not set
normal

Tracking

(Not tracked)

People

(Reporter: cavin, Unassigned)

References

(Depends on 1 open bug)

Details

(Keywords: perf)

It seems that we should delay sidebar loading until after the password dialog is 
dismissed (either users login correctly or cancel the dialog) because the 
password dialog is modal and users can't click on the sidebar anyway.

The test result from the 12-25 commercial build indicates that this change will 
probably save us 0.2 seconds on startup (ie, the difference between starting up 
mailnews with and without sidebar is about 0.2 second). It's an unofficial 
number but it gives a good indication of how much improvement we'll get on this 
change.
Moving it to 0.9.8.
Keywords: perf
Target Milestone: --- → mozilla0.9.8
Blocks: 26455
this should not be the case if we add a newsgroup account.. cause it doesn't
display sidebar now, after newsgroup account creation. see attachment 64655 [details].
But I do get Sidebar to show content on hitting F9 twice as in bug 116094
sliding to 0.9.9
Target Milestone: mozilla0.9.8 → mozilla0.9.9
we'll work on this later.
Target Milestone: mozilla0.9.9 → mozilla1.2
Blocks: 123205
I can reproduce this bug,but i think it's shouldn't mailnews bug,because i can 
reproduce this bug ,when i open a web page (authorization required),sidebar can 
not load.you can see follow code in extensions/wallet/src/nsWalletService.cpp

NS_IMETHODIMP
nsSingleSignOnPrompt::PromptPassword(const PRUnichar *dialogTitle, const 
PRUnichar *text, 
                                     const PRUnichar *passwordRealm, PRUint32 
savePassword, 
                                     PRUnichar **pwd, PRBool *_retval)
{
  nsresult rv;
  nsCAutoString realm;
  realm.AssignWithConversion(passwordRealm);     // XXX should be PRUnichar*
  rv = SINGSIGN_PromptPassword(dialogTitle, text, pwd,
                               realm.get(), mPrompt, _retval, savePassword);
  return rv;
}
Updating target milestone.
Target Milestone: mozilla1.2alpha → ---
Depends on: 178003
Product: Browser → Seamonkey
Assignee: sspitzer → mail
irrelevant today -> WONTFIX per bug 26455 comment 40.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → WONTFIX
> irrelevant today -> WONTFIX per bug 26455 comment 40.

1. MailNews Sidebar will come back for SeaMonkey.
2. Comment #6 sees this for non-mail as well (but that probably should be another bug).
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Status: REOPENED → NEW
QA Contact: esther
You need to log in before you can comment on or make changes to this bug.