[bugzilla-gnome-org-extensions] IE compat use a colgroup element



commit ed518160da369a2bf0dd8b5eb370f76a95e88342
Author: Owen W. Taylor <otaylor fishsoup net>
Date:   Sat Oct 10 17:36:49 2009 -0400

    IE compat use a colgroup element
    
    Setting widths on col elements didn't work in IE7 until
    they were put inside a colgroup element.

 js/splinter.js |   13 +++++++------
 1 files changed, 7 insertions(+), 6 deletions(-)
---
diff --git a/js/splinter.js b/js/splinter.js
index 1c80aa2..bfd9bb3 100644
--- a/js/splinter.js
+++ b/js/splinter.js
@@ -556,18 +556,19 @@ function onRowDblClick(e) {
 }
 
 function appendPatchTable(type, maxLine, parentDiv) {
-    var q = $("<table class='file-table'>"
+    var q = $("<table class='file-table'><colgroup></colgroup>"
               + "</table>").appendTo(parentDiv);
+    var colQ = q.find("colgroup");
     if (type != Patch.ADDED) {
-        q.append("<col class='line-number-column'></col>");
-        q.append("<col class='old-column'></col>");
+        colQ.append("<col class='line-number-column' span='1'></col>");
+        colQ.append("<col class='old-column' span='1'></col>");
     }
     if (type == Patch.CHANGED) {
-        q.append("<col class='middle-column'></col>");
+        colQ.append("<col class='middle-column' span='1'></col>");
     }
     if (type != Patch.REMOVED) {
-        q.append("<col class='line-number-column'></col>");
-        q.append("<col class='new-column'></col>");
+        colQ.append("<col class='line-number-column' span='1'></col>");
+        colQ.append("<col class='new-column' span='1'></col");
     }
 
     if (type == Patch.CHANGED)


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]