account central: no status bar if selection is on mail account

VERIFIED FIXED in mozilla0.8

Status

SeaMonkey
MailNews: Message Display
--
minor
VERIFIED FIXED
17 years ago
13 years ago

People

(Reporter: Oliver Klee, Assigned: racham)

Tracking

Trunk
mozilla0.8
x86
Windows 2000

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments)

(Reporter)

Description

17 years ago
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; m18) Gecko/20010125
BuildID:    2001012504

If the selection is on a mail account, there is no status bar at the bottom.

Reproducible: Always

Steps to Reproduce:
1. In MailNews folder tree, select an account.
2. Press "Get Messages".

Actual Results:  After selecting the accoung, a menu gets displayed: Mozilla
mail for <account name>: Email - read my Email messages ... and so on.

There is no status bar, so that neither the progress bar nor the usual "Host
contacting, sending login information" stuff gets displayed, although it is
possible to receive new email messages.


Expected Results:  The menu and a status bar at the bottom.

Comment 1

17 years ago
Same here. Searched for dup and couldn't find any.
Marking as new.

Mozilla 2001012420 on Windows 2000 SP1 on PC.
Status: UNCONFIRMED → NEW
Ever confirmed: true
over to racham.
Assignee: sspitzer → racham
Summary: no status bar if selection is on mail account → account central: no status bar if selection is on mail account

Updated

17 years ago
QA Contact: esther → nbaca
(Assignee)

Comment 3

17 years ago
Accepting. Easy to fix. Will post a patch soon (if my depend build ever gets
finished).
Status: NEW → ASSIGNED
(Assignee)

Comment 4

17 years ago
patch coming up..
Target Milestone: --- → mozilla0.8
(Assignee)

Comment 5

17 years ago
Created attachment 23751 [details] [diff] [review]
ptach to keep the status bar intact when AcctCentral page is on
(Assignee)

Updated

17 years ago
Keywords: nsbeta1, patch, review
(Assignee)

Comment 6

17 years ago
Prior to the patch posted..we had threadpane, splitter, messagepane and the
status bar all represented by one box (named 'messageBox'). That box was hid
when we displayed accountcentral page causing status bar also to disappear. So,
I have introduced a new box for status bar and moved it out of 'messageBox'.
That puts us in the right place.

In the patch, I have also touched couple of other lines lacking proper
indentation (the whole box system of 3pane window).  
please provide a new version of this, using cvs diff -uw.

you can check in white space changes, but there isn't any reason for me to
review them.
(Assignee)

Comment 8

17 years ago
Created attachment 23756 [details] [diff] [review]
new version diffs (with -uw)
do you need to do anything for the vertical 3 pane layout xul?
adding myself to the cc list.
(Assignee)

Comment 11

17 years ago
There is a separate bug for that one (bug 66442). I will be fixing that soon. 
sr=sspitzer
(Assignee)

Comment 13

17 years ago
Fix checked in. Thanks Seth.

Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
bhuvan, 

I had to back you out.  bryner found problems (bryner, can you attach that
screen shot) and I had problems with resizing the three pane after your checkin.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Created attachment 23785 [details]
screenshot of mail window after this checkin
on linux, my problem was this:

on resize of the 3 pane, I'd lose the status bar completely.

bryner's problems were on Windows (correct me if I'm wrong)
(Assignee)

Comment 17

17 years ago
Sorry about that...I will rework on the patch.
(Assignee)

Comment 18

17 years ago
New patch coming patch.

It is even more simpler. I just need to move the status bar panel out of this
box system (which the way it was and should be now also) so that it just sits
where it is supposed to be without trying to encroach..!

Tested on Windows and Linux. Resizing the window and expand/collapse with
splitter. Works fine.
(Assignee)

Comment 19

17 years ago
Created attachment 24020 [details] [diff] [review]
updated patch
(Assignee)

Comment 20

17 years ago
Seth,

One more review please..?
I tried it on linux, too.  no problems.  looks good.  sr=sspitzer

(Assignee)

Comment 22

17 years ago
Fixed for real.
Status: REOPENED → RESOLVED
Last Resolved: 17 years ago17 years ago
Resolution: --- → FIXED

Comment 23

17 years ago
Build 2001-02-19-08: NT4
Build 2001-02-19-04: Linux RH 6.2
Build 2001-02-22-08: Mac 9.04
Verified Fixed, 

Status Bar now appears when Account Central:
- is displayed by itself
- is displayed and the "Get Msg" button is selected
- is displayed and the Read Messages link is selected
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.