From 1e51410de5b53a0259452540e8485947d73e56f0 Mon Sep 17 00:00:00 2001 From: "naving%netscape.com" Date: Mon, 8 Jan 2001 19:17:42 +0000 Subject: [PATCH] 52564, r=naving sr=mscott. Fixing the new Folder dialog issue and taking care of long folder names without the use of window.sizeToContent(). Thanks to disttsc@bart.nl for providing this patch --- mailnews/base/resources/content/msgFolderPickerOverlay.js | 7 ------- mailnews/base/resources/content/newFolderDialog.js | 8 +++----- mailnews/base/resources/content/newFolderDialog.xul | 3 ++- 3 files changed, 5 insertions(+), 13 deletions(-) diff --git a/mailnews/base/resources/content/msgFolderPickerOverlay.js b/mailnews/base/resources/content/msgFolderPickerOverlay.js index 9b4ee0adabe9..6f0d37f302ca 100644 --- a/mailnews/base/resources/content/msgFolderPickerOverlay.js +++ b/mailnews/base/resources/content/msgFolderPickerOverlay.js @@ -97,11 +97,4 @@ function SetFolderPicker(uri,pickerID) picker.setAttribute("value",selectedValue); picker.setAttribute("uri",uri); - - if (pickerID == "msgNewFolderPicker") { - // XXX Ugly hack to make sizeToContent work, see bug 62987 - window.resizeTo(0,0); - - window.sizeToContent(); - } } diff --git a/mailnews/base/resources/content/newFolderDialog.js b/mailnews/base/resources/content/newFolderDialog.js index 9087cecf2d24..ab74871a7562 100644 --- a/mailnews/base/resources/content/newFolderDialog.js +++ b/mailnews/base/resources/content/newFolderDialog.js @@ -48,19 +48,17 @@ function onLoad() if (arguments.dualUseFolders) { dialog.folderType = FOLDERS | MESSAGES; } else { + window.resizeTo(250, 235); + // set our folder type by calling the default selected type's oncommand var selectedFolderType = document.getElementById("folderGroup").selectedItem; eval(selectedFolderType.getAttribute("oncommand")); // show the section which allows us to select the folder type to create var newFolderTypeBox = document.getElementById("newFolderTypeBox"); - newFolderTypeBox.setAttribute("hidden", "false"); + newFolderTypeBox.removeAttribute("hidden"); } - // XXX Ugly hack to make sizeToContent work, see bug 62987 - window.resizeTo(0, 0); - - window.sizeToContent(); moveToAlertPosition(); doEnabling(); doSetOKCancel(onOK, onCancel); diff --git a/mailnews/base/resources/content/newFolderDialog.xul b/mailnews/base/resources/content/newFolderDialog.xul index 7d87539cfdde..a495499007ed 100644 --- a/mailnews/base/resources/content/newFolderDialog.xul +++ b/mailnews/base/resources/content/newFolderDialog.xul @@ -34,6 +34,7 @@ title="&newFolderDialog.title;" align="vertical" class="dialog" + width="200" height="180" onload="onLoad();">