Issue mozilla#25 - addressed comments, part 2:

- move <prefwindow> styling to titlebar.css - avoid defaultView.window - no space between named function and "("
xabolcs · Jul 9, 2012 · 66c0392 · 66c0392
1 parent ba06511
commit 66c0392
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/extension/chrome/content/titlebar/customize.js b/extension/chrome/content/titlebar/customize.js
@@ -42,7 +42,7 @@ function arrayBasedTreeView (treeViewData) {
 arrayBasedTreeView.prototype = {
   data: [],
 
-  get rowCount () {
+  get rowCount() {
     return this.data.length;
   },
 
@@ -95,7 +95,7 @@ variables: [],
 
 init: function(aEvent)
 {
-  aEvent.originalTarget.defaultView.window.removeEventListener("load", paneTitle.init, false);
+  aEvent.originalTarget.defaultView.removeEventListener("load", paneTitle.init, false);
 
   var mediator = Components.classes['@mozilla.org/appshell/window-mediator;1']
               .getService(Components.interfaces.nsIWindowMediator);      

diff --git a/extension/chrome/content/titlebar/customize.xul b/extension/chrome/content/titlebar/customize.xul
@@ -43,7 +43,7 @@
 
 <prefwindow id="NightlyTesterOptions" windowtype="NightlyTester:Customize"
             title="&nightly.customize.title;"
-            style="width: 42.75em; height: 28.5em" persist="width height"
+            persist="width height"
             xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
 
   <prefpane id="paneTitle" label="&nightly.customize.title;" flex="1">

diff --git a/extension/chrome/skin/titlebar/titlebar.css b/extension/chrome/skin/titlebar/titlebar.css
@@ -38,3 +38,6 @@
 #variableTree {
   height: 17em;
 }
+#NightlyTesterOptions {
+  width: 42.75em;
+}