From 085857b4ab516972262c4cd3ad2455fa98f6e560 Mon Sep 17 00:00:00 2001
From: Roland Warmerdam <rowno@webspirited.com>
Date: Thu, 7 Oct 2010 10:28:04 -0700
Subject: [PATCH] Bug 592708: Hide the bottom border of the details grid.
 r=Unfocused, a=dtownsend

---
 toolkit/mozapps/extensions/content/extensions.js       | 10 ++++++++++
 .../gnomestripe/mozapps/extensions/extensions.css      |  7 ++++++-
 .../themes/pinstripe/mozapps/extensions/extensions.css |  7 ++++++-
 .../themes/winstripe/mozapps/extensions/extensions.css |  7 ++++++-
 4 files changed, 28 insertions(+), 3 deletions(-)

diff --git a/toolkit/mozapps/extensions/content/extensions.js b/toolkit/mozapps/extensions/content/extensions.js
index 67c52744ad2..a3a215b036f 100644
--- a/toolkit/mozapps/extensions/content/extensions.js
+++ b/toolkit/mozapps/extensions/content/extensions.js
@@ -1997,6 +1997,16 @@ var gDetailView = {
     }
 
     document.getElementById("detail-prefs").hidden = !aIsRemote && !aAddon.optionsURL;
+    
+    var gridRows = document.querySelectorAll("#detail-grid rows row");
+    for (var i = 0, first = true; i < gridRows.length; ++i) {
+      if (first && window.getComputedStyle(gridRows[i], null).getPropertyValue("display") != "none") {
+        gridRows[i].setAttribute("first-row", true);
+        first = false;
+      } else {
+        gridRows[i].removeAttribute("first-row");
+      }
+    }
 
     this.updateState();
 
diff --git a/toolkit/themes/gnomestripe/mozapps/extensions/extensions.css b/toolkit/themes/gnomestripe/mozapps/extensions/extensions.css
index f996a09430d..817c32f08cb 100644
--- a/toolkit/themes/gnomestripe/mozapps/extensions/extensions.css
+++ b/toolkit/themes/gnomestripe/mozapps/extensions/extensions.css
@@ -605,9 +605,14 @@
   margin-bottom: 2em;
 }
 
+.detail-row[first-row="true"],
+.detail-row-complex[first-row="true"] {
+  border-top: none;
+}
+
 .detail-row,
 .detail-row-complex {
-  border-bottom: 1px solid grey;
+  border-top: 1px solid grey;
   -moz-box-align: center;
 }
 
diff --git a/toolkit/themes/pinstripe/mozapps/extensions/extensions.css b/toolkit/themes/pinstripe/mozapps/extensions/extensions.css
index b2cd3e8a4e2..fdef27a6b49 100644
--- a/toolkit/themes/pinstripe/mozapps/extensions/extensions.css
+++ b/toolkit/themes/pinstripe/mozapps/extensions/extensions.css
@@ -656,9 +656,14 @@
   margin-bottom: 2em;
 }
 
+.detail-row[first-row="true"],
+.detail-row-complex[first-row="true"] {
+  border-top: none;
+}
+
 .detail-row,
 .detail-row-complex {
-  border-bottom: 1px solid grey;
+  border-top: 1px solid grey;
   -moz-box-align: center;
 }
 
diff --git a/toolkit/themes/winstripe/mozapps/extensions/extensions.css b/toolkit/themes/winstripe/mozapps/extensions/extensions.css
index 6c704530ede..92c5bfd4ad6 100644
--- a/toolkit/themes/winstripe/mozapps/extensions/extensions.css
+++ b/toolkit/themes/winstripe/mozapps/extensions/extensions.css
@@ -641,9 +641,14 @@
   margin-bottom: 2em;
 }
 
+.detail-row[first-row="true"],
+.detail-row-complex[first-row="true"] {
+  border-top: none;
+}
+
 .detail-row,
 .detail-row-complex {
-  border-bottom: 1px solid grey;
+  border-top: 1px solid grey;
   -moz-box-align: center;
 }