2011-01-11 Ojan Vafai <ojan@chromium.org>

        Reviewed by Adam Barth.

        fix difflink centering to be vertical, not horizontal
        https://bugs.webkit.org/show_bug.cgi?id=52263

        * PrettyPatch/PrettyPatch.rb:
        * code-review.js:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@75571 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/Websites/bugs.webkit.org/ChangeLog b/Websites/bugs.webkit.org/ChangeLog
index 18f13d6addfd58d1c4a1100117affe3c8372a5c4..8d9df95cda3c9e83c9c4d04775fff82001f5a94a 100644 (file)
--- a/Websites/bugs.webkit.org/ChangeLog
+++ b/Websites/bugs.webkit.org/ChangeLog
@@ -1,3 +1,13 @@
+2011-01-11  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Adam Barth.
+
+        fix difflink centering to be vertical, not horizontal
+        https://bugs.webkit.org/show_bug.cgi?id=52263
+
+        * PrettyPatch/PrettyPatch.rb:
+        * code-review.js:
+
 2011-01-11  Ojan Vafai  <ojan@chromium.org>
 
         Reviewed by Adam Barth.

diff --git a/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb b/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
index 1cc150fc91417ecfe20fca296378adf54604c1f4..ac2c69e531c5066ab3d4f887866f8e24e9253ab8 100644 (file)
--- a/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
+++ b/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
@@ -121,6 +121,9 @@ h1 {
     font-family: sans-serif;
     font-size: 1em;
     margin-left: 0.5em;
+    display: table-cell;
+    width: 100%;
+    padding: 0.5em;
 }
 
 h1 :link, h1 :visited {
@@ -150,7 +153,9 @@ h1 :hover {
 }
 
 .FileDiffLinkContainer {
-    text-align: center;
+    display: table-cell;
+    padding-right: 0.5em;
+    white-space: nowrap;
 }
 
 .DiffSection {

diff --git a/Websites/bugs.webkit.org/code-review.js b/Websites/bugs.webkit.org/code-review.js
index b7f19737d9886fd013f3c5913ce5bfa625a782cf..e6ae2200ca4ef367839c643c4721329b7fcfc8ae 100644 (file)
--- a/Websites/bugs.webkit.org/code-review.js
+++ b/Websites/bugs.webkit.org/code-review.js
@@ -314,7 +314,7 @@
       var file_name = $(this).children('h1').text();
       files[file_name] = this;
       addExpandLinks(file_name);
-      $('h1', this).before('<div class="FileDiffLinkContainer">' + diffLinksHtml() + '</div>');
+      $('h1', this).after('<div class="FileDiffLinkContainer">' + diffLinksHtml() + '</div>');
       updateDiffLinkVisibility(this);
     });
   }