Bug 662051 - Port |Bug 608934 - About window needs a close button| to comm-central; r=Standard8
authorTakanori MATSUURA <t.matsuu@gmail.com>
Sun, 05 Jun 2011 19:26:12 +0900
changeset 8262 a264f7b230164354885c6aa49b90ac2a969b16f6
parent 8261 30308622251ceb232ef938c6d1ebabf56a11cdf1
child 8263 9dc27ff100db032d2ed99f83e8596cf5fd4e6c4f
push id84
push userbugzilla@standard8.plus.com
push dateTue, 16 Aug 2011 21:25:04 +0000
treeherdercomm-beta@6970c86be3cd [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersStandard8
bugs662051, 608934
Bug 662051 - Port |Bug 608934 - About window needs a close button| to comm-central; r=Standard8
mail/base/content/mailCore.js
--- a/mail/base/content/mailCore.js
+++ b/mail/base/content/mailCore.js
@@ -467,21 +467,22 @@ function openAboutDialog()
   let enumerator = Services.wm.getEnumerator("Mail:About");
   while (enumerator.hasMoreElements()) {
     // Only open one about window
     let win = enumerator.getNext();
     win.focus();
     return;
   }
 
-#ifdef XP_MACOSX
+#ifdef XP_WIN
+  var features = "chrome,centerscreen,dependent";
+#elifdef XP_MACOSX
   var features = "chrome,resizable=no,minimizable=no";
 #else
-  // XXX Should have dependent as well?
-  var features = "chrome,centerscreen,resizable=no";
+  var features = "chrome,centerscreen,dependent,dialog=no";
 #endif
   window.openDialog("chrome://messenger/content/aboutDialog.xul", "About", features);
 }
 
 /**
  * Opens the support page based on the app.support.baseURL pref.
  */
 function openSupportURL()