53795 - switching skins results in data loss in composer

Status: VERIFIED DUPLICATE of bug 43350

(Core Graveyard :: Skinability, defect, P1)

Description

[sujay]

using 9/22 build of netscape

1) launch netscape
2) launch composer

Modern skin is what comes up as my default

3) enter some text/objects
4) using Prefs, change the skin to classic and hit Apply.

notice all the text/objects you entered into Composer is gone.

Comment 1

[sujay]

over to Ben Goodger

Comment 2

[Charles Manske]

Isn't this a dup of 57394 (or vice versa)?

Comment 3

[sujay]

this is not a DUP....53794 is about having to exit composer and relaunch  in
order to even use it

this bug is about data loss..

Comment 4

[Charles Manske]

Yes, but the problem is the same, you really have to  quit composer (including
saving your changes) before switching skins.
This may be the simplest fix: loop through all open Composer windows and 
do "onclose", which will force users to save data in each window, then 
continue to switch skins. Maybe even save the url during the closing loop and
reopen windows after switching skin?

Comment 5

[rubydoo123]

nominating for nsbeta3, we encounter dataloss in composer and in mail compose

Comment 6

[don]

Marking this as P1 [nsbeta3-] and [rtm+]. I wouldn't hold beta for it, but we
better fix this before we ship.

Ben, is this our bug or something for hyatt?  And is it related to bug #53794?

Comment 7

[Charles Manske]

Attachment: A JavaScript method to close all HTML composer windows

Comment 8

[Charles Manske]

Use the attached method to loop through all HTML Composer windows (including
mail). This probably needs extra logic to properly close down a mail message
window (call a mail-specific routine instead of 
tempWindow.editorShell.CloseWindowWithoutSaving) or maybe a different 'CanClose'
method, but this will get you started. This should be executed from a browser
window, not from one of the windows being closed.
Note that you have to use EditorCanClose() to properly close the Insert 
Character non-modal dialog window that is associated with an open Composer 
window.

Comment 9

[Ben Goodger (use ben at mozilla dot org for email)]

these bugs will not be fixed before N6.0

Comment 10

[Ben Goodger (use ben at mozilla dot org for email)]

er, rtm-

Comment 11

[Simon Fraser [no longer active]]

*** Bug 53794 has been marked as a duplicate of this bug. ***

Comment 12

[Simon Fraser [no longer active]]

Ben, your manager has to do the +ing and -ing. Putting don's rtm+ back.

Comment 13

[Simon Fraser [no longer active]]

Dup of 43350


*** This bug has been marked as a duplicate of 43350 ***

Comment 14

[Scott Brodmerkle]

Verified dupe.