From 05d855a48dd62646edc502cc3d10abedaea19f3d Mon Sep 17 00:00:00 2001
From: "dmose%netscape.com" <dmose%netscape.com>
Date: Wed, 20 Nov 2002 02:09:34 +0000
Subject: [PATCH] Initial size of junk mail dialog is huge (bug 180215).  Patch
 from neil@parkwaycc.co.uk; thanks Neil!  r=dmose, sr=sspitzer

---
 mailnews/base/resources/content/junkMail.js  | 10 ++----
 mailnews/base/resources/content/junkMail.xul | 35 ++++++++------------
 2 files changed, 16 insertions(+), 29 deletions(-)

diff --git a/mailnews/base/resources/content/junkMail.js b/mailnews/base/resources/content/junkMail.js
index 7a9b942b4513..e54d3bb1cb82 100644
--- a/mailnews/base/resources/content/junkMail.js
+++ b/mailnews/base/resources/content/junkMail.js
@@ -1,15 +1,14 @@
 var gSpamSettings = {};
 var gCurrentServer;
-var gMessengerBundle = document.getElementById("bundle_messenger");
+var gMessengerBundle;
 
 function onJunkMailLoad()
 {
+  gMessengerBundle = document.getElementById("bundle_messenger");
   if (window.arguments && window.arguments[0]) {
     // XXX todo, what if no folder?
     setupForAccountFromFolder(window.arguments[0].folder.URI);
   }
-
-  doSetOKCancel(onAccept, onCancel, doHelpButton);
 }
 
 function onServerClick(event)
@@ -184,8 +183,3 @@ function doHelpButton()
 
   return false; // don't close the top-level window
 }
-
-function onCancel()
-{
-  return true; // close the top-level window
-}
diff --git a/mailnews/base/resources/content/junkMail.xul b/mailnews/base/resources/content/junkMail.xul
index 4031cb17fafb..36a3e35cd293 100644
--- a/mailnews/base/resources/content/junkMail.xul
+++ b/mailnews/base/resources/content/junkMail.xul
@@ -2,29 +2,28 @@
 
 <?xml-stylesheet href="chrome://messenger/skin/accountManage.css" type="text/css"?>
 
-<?xul-overlay href="chrome://global/content/dialogOverlay.xul"?>
-
 <?xul-overlay href="chrome://messenger/content/msgFolderPickerOverlay.xul"?>
 
 <!DOCTYPE window SYSTEM "chrome://messenger/locale/junkMail.dtd">
 
-<window id="junkMail"
+<dialog id="junkMail"
         xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
         xmlns:nc="http://home.netscape.com/NC-rdf#"
         xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
         windowtype="mailnews:junk"
         title="&window.title;"
-        onload="onJunkMailLoad()"
-        persist="screenX screenY width height">
+        onload="onJunkMailLoad();"
+        buttons="accept,cancel,help"
+	ondialogaccept="return onAccept();"
+	ondialoghelp="return doHelpButton();">
 
-<stringbundleset id="stringbundleset">
-  <stringbundle id="bundle_messenger" src="chrome://messenger/locale/messenger.properties"/>
-</stringbundleset>
+  <script type="application/x-javascript" src="chrome://messenger/content/junkMail.js"/>
+  <script type="application/x-javascript" src="chrome://messenger/content/widgetglue.js"/>
 
-<script type="application/x-javascript" src="chrome://messenger/content/junkMail.js"/>
-<script type="application/x-javascript" src="chrome://messenger/content/widgetglue.js"/>
+  <stringbundleset id="stringbundleset">
+    <stringbundle id="bundle_messenger" src="chrome://messenger/locale/messenger.properties"/>
+  </stringbundleset>
 
-<vbox>
   <hbox align="center">
     <label value="&account.label;" accesskey="&account.accesskey;"/>
     <menulist oncommand="onServerClick(event);" id="server">
@@ -47,8 +46,8 @@
     <button label="&viewLog.label;" accesskey="&viewLog.accesskey;" oncommand="junkLog()"/>
   </hbox>
 
-  <description>&info1.label;</description>
-  <description>&info2.label;</description>
+  <description width="1">&info1.label;</description>
+  <description width="1">&info2.label;</description>
 
   <checkbox id="level" oncommand="conditionallyEnableUI(null);" accesskey="&level.accesskey;" label="&level.label;"/>
 
@@ -96,7 +95,7 @@
       </grid>
     </radiogroup>
     
-    <hbox autostretch="never" align="center">
+    <hbox align="center">
       <spacer width="20px"/>
       <checkbox id="purge" oncommand="conditionallyEnableUI('purge')" accesskey="&purge1.accesskey;" label="&purge1.label;"/>
       <textbox size="3" id="purgeInterval" />
@@ -104,10 +103,4 @@
     </hbox>
   </vbox>
 
-  <separator/>
-
-  <hbox id="okCancelHelpButtonsRight"/>
-
-</vbox>
-
-</window>
+</dialog>