90691 - Mail Compose: Remove "Done loading page" from statusbar

Status: VERIFIED FIXED

(MailNews Core :: Composition, defect)

Description

[jglick]

When you open a new Mail Compose window, the text "Document: Done" 
following by "Done loading page" appears on the statusbar.  Can we remove this 
text from the statusbar?  Done loading what? Document Done doing what?  Users 
don't think of mail compose as needing to "load content".

The statusbar should be blank unless there useful information about the task at 
hand to help the user, such as when the message is being sent.

Comment 1

[jglick]

Attachment: Example

Comment 2

[Jean-Francois Ducarroz]

-->varada. I think editor put this text on the status bar.

Comment 3

[jglick]

nominating. If we want users to look at the status bar for useful info, we 
should use it that way.

Comment 4

[[e-mail] (gone - use [e-mail] instead)]

This is because we inherit editor.xul and TextEditorAppShell.xul, which both
call editor.dtd which has in line 45 a label "Done loading page".  I can't
figure out how to override that, but I did manage to whack "Document: Done".

Comment 5

[[e-mail] (gone - use [e-mail] instead)]

Attachment: Fix.

Comment 6

[scottputterman]

marking nsbeta1-, but moving to 0.9.9 to get the patch in.

Comment 7

[Håkan Waara]

Comment on attachment 54841 [details] [diff] [review]
Fix.

If Seth is OK with this partial fix, I guess I am too.	After all, it improves
the behaviour a little.

r=hwaara

Comment 8

[Joe Hewitt (gone)]

Comment on attachment 54841 [details] [diff] [review]
Fix.

sr=hewitt

Comment 9

[[e-mail] (gone - use [e-mail] instead)]

Partial fix landed.  The summary is correct as it stands for the remaining 
portion.

Comment 10

[Koike Kazuhiko]

Attachment: patch

doesn't show "Done loading page" on mail compose window.

Comment 11

[[e-mail] (gone - use [e-mail] instead)]

Attachment: Screenshot, proof this works.

I ran Mozilla after building this simple (but effective) patch, and I'm happy
to report that it works fine.

I've also sent mail privately to Kathleen Brade for review.

Comment 12

[Charles Manske]

Comment on attachment 75315 [details] [diff] [review]
patch

r=cmanske
This is temporary anyway, since nsEditorShell.cpp is going away. The status
line in Composer will be fixed by bug 47519

Comment 13

[Simon Fraser [no longer active]]

Comment on attachment 75315 [details] [diff] [review]
patch

sr=sfraser

Comment 14

[[e-mail] (gone - use [e-mail] instead)]

Fix checked in by Charles Manske, thanks!

Comment 15

[[e-mail] (gone - use [e-mail] instead)]

[e-mail], thanks for your patch.  It's been checked in.  Marking FIXED.

Comment 16

[[e-mail] (gone - use [e-mail] instead)]

Mac OS 9.2.2 - 2002-09-24-08
Windows 2000 - 2002-09-24-08
Mac OS X 10.1.5 - 2002-09-24-08
RedHat 8.0 - 2002-09-24-08

Tested both Classic and Modern.

Verified FIXED