Closed Bug 80511 Opened 24 years ago Closed 23 years ago

Bad defaults for initial 3-pane size and positioning

Categories

(SeaMonkey :: MailNews: Message Display, defect, P4)

x86
All

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.9.3

People

(Reporter: hwaara, Assigned: hwaara)

Details

Attachments

(4 files)

<RDF:Description about="chrome://messenger/content/messenger.xul#messengerWindow"> <screenX>0</screenX> <screenY>191</screenY> <width>750</width> <height>500</height> </RDF:Description> That is from the default localstore.rdf that provides the default positions and sizes for new profiles. Since we want Mozilla to work in 640x480 mode, we can't just let 3pane be 110x211 pixels offscreen at startup. That's just wrong, and it will give the user a bad first experience with mailnews. So I propose this: let's modify the values to give the user a window that is onscreen and a reasonable size at the first startup.
This is worse than some people might think, especially for those with small screens. I know mpt whined about this in #mozilla today for example...
Keywords: nsbeta1
Priority: -- → P1
Target Milestone: --- → mozilla0.9.1
bumping off the 0.9.1 list
Priority: P1 → P4
Target Milestone: mozilla0.9.1 → mozilla0.9.2
QA Contact: esther → nbaca
Mpt: I need coordinates.
Same as for the browser, bug 68941: that is, origin at (4,4), and dimensions of (632,450). (Don't bother trying to share these dimensions between components, because they'll start being different once we have smarter window geometry.)
Rebuilding and testing a possible fix for this right now...
Status: NEW → ASSIGNED
The latest screenshot is how it looks for me after my fix. I created a new profile, and just clicked the mailnews button once I saw the first initial browser window so this is how it looks for a first time user. Note that the Account Wizard hides the 3-pane a bit (I think that one is a bit too large), but please ignore that in this bug. This change looks good to me, I will file a bug about the Account Wizars sizing though. Thoughts?
Tried it in 640x480, and it worked very nice compared to before. Requesting r= and sr= on the upcoming patch.
Attached patch fix for this and bug 68941 (deleted) — Splinter Review
If you r=/sr= this little fix, then you might was well go ahead and review the browser-change included in the patch and note it in bug 68941. Thanks!
It's the same patch as in bug 68941. If I were anal retentive at all, this would make me bark. Phbbbbb. OK, r=danm, visiting from that other bug.
CC alecf for sr=. Alec, this is the same patch as the other bug but is about the other part of the patch. It's a really small code change.
sr=alecf
what about for the std alone message window and the vertical (alternate) 3 pane window?
Seth, please file separate bugs on me if you want them fixed too. I'll gladly fix them. :)
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Build 2001-06-26-03: WinMe What should I expect to see? Should the entire window be in view and not cut-off? If this is the case then this bug should be reopened. I created a new profile and launched mail from the browser window (the browser window is similar to the mail window). - 640x480: The top left portion of the mail window appears, the entire window does not fit in the screen. - 800x600: Similar results
Attached image View of how it looks w/800x600 (deleted) —
Nbaca, a few questions: 1. That's from a new profile where you haven't move the 3pane, right? (just checking) 2. Can you try this with a Mozilla nightly? May be something NS-specific.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: mozilla0.9.2 → mozilla0.9.3
1. Yes, it was in a new profile and I didnt' move the 3pane. 2. Yes, I'll try a mozilla build.
Using a build from 2001-06-27, the Mozilla build looks good in 640x480 and 800x600! Trunk Commercial build 2001-06-27: doesn't appear to have the fix. Branch Commercial build 2001-06-27-08: doesn't appear to have the fix. Does anyone know when this fix will appear, atleast on the trunk?
there is probably a commercial-only localstore.rdf that needs to be modified..
I created Bugscape bug# 6898 to cover the commercial problem. Marking this bug Resolved/Fixed.
Status: REOPENED → RESOLVED
Closed: 23 years ago23 years ago
Resolution: --- → FIXED
Verified Fixed.
Status: RESOLVED → VERIFIED
Great, thanks.
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: