From 66c039292aa9b82b84b1e1e392f33d8930e9234c Mon Sep 17 00:00:00 2001 From: Szabolcs Hubai Date: Mon, 9 Jul 2012 23:24:27 +0200 Subject: [PATCH] Issue #25 - addressed comments, part 2: - move styling to titlebar.css - avoid defaultView.window - no space between named function and "(" --- extension/chrome/content/titlebar/customize.js | 4 ++-- extension/chrome/content/titlebar/customize.xul | 2 +- extension/chrome/skin/titlebar/titlebar.css | 3 +++ 3 files changed, 6 insertions(+), 3 deletions(-) diff --git a/extension/chrome/content/titlebar/customize.js b/extension/chrome/content/titlebar/customize.js index bc846ea..ffbadfb 100644 --- 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 index ecffaea..831baca 100644 --- a/extension/chrome/content/titlebar/customize.xul +++ b/extension/chrome/content/titlebar/customize.xul @@ -43,7 +43,7 @@ diff --git a/extension/chrome/skin/titlebar/titlebar.css b/extension/chrome/skin/titlebar/titlebar.css index 44b4b56..f35f890 100644 --- 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; +}