24019 - Compose window size is not remembered

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect, P3)

Description

[scalkins]

All platforms:
Win build 2000-01-12-11 M13 (NT)
Linux 2000-01-11-10 M13 (RH 6.0)
Mac 2000-01-12-08 M13 (Mac OS 8.6)

Steps to reproduce:
1) Launch Seamonkey.
2) Open the Composer component via the Tasks menu.
3) Resize Compose window so it's very tiny, very
slim, etc...such that you would notice if the change "took".
4)Either:
   a)Close the components window by dismissing it's window (i.e. via
     File -->Close, or clicking on the X in upper right corner of window)or
   b)Exit Seamonkey via File-->Quit
5)a)If you simply closed the component window, reopen it via the Tasks menu.
  b)If you closed seamonkey, relaunch Seamonkey and open the same components to
    see if the window sizes are the same.

Expected results: Window sizes stay at the changed size when Compose is
re-opened.

Actual results:
Composer window size resets to defaults.

Comment 1

[rubydoo123]

assigning this to brade for debugging -- reassign as appropriate

Comment 2

[rubydoo123]

forgot to change owner - doing that now.

Comment 3

[Kathleen :Brade]

Here is the diff for this fix:

Index: editor.xul
===================================================================
RCS file: /cvsroot/mozilla/editor/ui/composer/content/editor.xul,v
retrieving revision 1.19
diff -c -3 -r1.19 editor.xul
*** editor.xul	2000/01/24 15:30:04	1.19
--- editor.xul	2000/01/24 22:16:04
***************
*** 42,48 ****
          titlemenuseparator="&editorWindow.titlemodifiermenuseparator;"
          windowtype="composer:html"
          align="vertical"
!         width="640" height="480">

    <html:script language="JavaScript"
src="chrome://editor/content/EditorCommands.js"/>
    <html:script language="JavaScript"
src="chrome://editor/content/EditorCommandsDebug.js"/>
--- 42,51 ----
          titlemenuseparator="&editorWindow.titlemodifiermenuseparator;"
          windowtype="composer:html"
          align="vertical"
!         width="640" height="480"
!         screenX="10" screenY="10"
!         persist="screenX screenY width height"
!         >

    <html:script language="JavaScript"
src="chrome://editor/content/EditorCommands.js"/>
    <html:script language="JavaScript"
src="chrome://editor/content/EditorCommandsDebug.js"/>

Comment 4

[Kathleen :Brade]

fixed; should be in today's builds

Comment 5

[sujay]

verified in 1/31 build.