Closed Bug 68345 Opened 24 years ago Closed 21 years ago

Don't show Mail Start Page before Account Central loads

Categories

(SeaMonkey :: MailNews: Message Display, defect)

All
Windows NT
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
Future

People

(Reporter: nbaca, Assigned: neil)

References

Details

(Keywords: polish)

Attachments

(1 file)

Build 2001-02-09-04: NT4 Overview: If Logon at Startup is selected, the 3pane layout with the Mail Start page flashes and then it displays the Account Central page. Steps to reproduce: 1. Select the "Log onto this account at Startup of Mail" 2. Exit, restart Mail Actual Results: 1. Flashing problem - The 3pane layout appears with the Mail Start page in the message pane. It displays current news of the day. - Quickly it flashes to the Account Central page which is strange, a novice user will wonder what they saw. Expected Results: If the Logon at Startup option is selected then Mail should automatically show the Account Central page. It would be much cleaner, less confusing.
Marking nsbeta1 because we don't want to confuse novice users. Thanks to Laurel for pointing this out.
Keywords: nsbeta1
QA Contact: esther → nbaca
Assigning to Bhuvan.
Assignee: sspitzer → racham
Correction. Step #1 should actually state: 1. Verify that the Logon at Startup option is Not selected. (This allows for the Account Central page to appear)
Bhuvan or Ninoschka, do these problems go away with Bhuvan's recent checkins to change the meaning of the startup pref?
The problem still exists for any given account that goes to AccountCentral first i.e., all those accounts for which 'Check for new mail at startup' option is unchecked. As the AccountCentral is getting triggered only after a folder is selected in the folder pane and that is little too late for message pane to not to display the startpage..So, I think we can simply clear the messagepane when we detect that we are going to display the account central page.
Status: NEW → ASSIGNED
Keywords: nsbeta1nsbeta1-
Target Milestone: --- → Future
marking nsbeta1-
Keywords: nsCatFood
*** Bug 117106 has been marked as a duplicate of this bug. ***
Would not loading the start page in the messenger onload fix this? There is already code to load the start page after the folder is loaded.
Attached patch Proposed PatchSplinter Review
When the default account is set to check for mail at start up, the start page still loads. When it is not set the check for mail at start up, the start page does not load. Note: testing was done with a start page of javascript:alert('Start');
Keywords: patch, polish, review
I talked to bhuvan on how to reproduce this, but I'm not seeing the problem under NT4 or win2k. nbaca, can you reproduce this problem with today's build?
Sean, the easiest way to reproduce this is to use an obvious start page - javascript:alert('Show the Start Page') for example - and turn off check for new mail at startup, of course.
mass re-assign.
Assignee: racham → sspitzer
Status: ASSIGNED → NEW
-> me.
Assignee: sspitzer → technutz
We already call this elsewhere, and I've verified that removing the call in msgMail3PaneWindow.js still enables loadStartPage() to be called (with my POP3 account): http://lxr.mozilla.org/seamonkey/source/mailnews/base/resources/content/commandglue.js#793 791 if (gDisplayStartupPage) 792 { 793 loadStartPage(); 794 gDisplayStartupPage = false; 795 UpdateMailToolbar("gDisplayStartupPage");
Assignee: technutz → neil.parkwaycc.co.uk
Comment on attachment 74553 [details] [diff] [review] Proposed Patch David, I'm not sure if I qualify to mark this as r=me, but I've tested this and it's worked fine with my POP3 account...
Attachment #74553 - Flags: review+
Attachment #74553 - Flags: superreview?(bienvenu) → superreview+
Fix checked in.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
*** Bug 61950 has been marked as a duplicate of this bug. ***
Verified FIXED
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: