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

        Reviewed by Adam Barth.

        fix wrapping bug in expansion lines in the code review tool
        https://bugs.webkit.org/show_bug.cgi?id=52270

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

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

diff --git a/Websites/bugs.webkit.org/ChangeLog b/Websites/bugs.webkit.org/ChangeLog
index 8d9df95cda3c9e83c9c4d04775fff82001f5a94a..75942ab6fb306e8e29448998813b096cbf4c1c07 100644 (file)
--- a/Websites/bugs.webkit.org/ChangeLog
+++ b/Websites/bugs.webkit.org/ChangeLog
@@ -1,3 +1,13 @@
+2011-01-12  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Adam Barth.
+
+        fix wrapping bug in expansion lines in the code review tool
+        https://bugs.webkit.org/show_bug.cgi?id=52270
+
+        * 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 ac2c69e531c5066ab3d4f887866f8e24e9253ab8..84b1ff2005ecb20e6f9b95d43e9f12f99fcb357f 100644 (file)
--- a/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
+++ b/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
@@ -319,6 +319,7 @@ body {
 
 .ExpandArea {
   margin: 0;
+  white-space: pre-wrap;
 }
 
 .ExpandText {

diff --git a/Websites/bugs.webkit.org/code-review.js b/Websites/bugs.webkit.org/code-review.js
index e6ae2200ca4ef367839c643c4721329b7fcfc8ae..6836be2db727b57c08c2bc9755331d1598b23245 100644 (file)
--- a/Websites/bugs.webkit.org/code-review.js
+++ b/Websites/bugs.webkit.org/code-review.js
@@ -350,7 +350,7 @@
 
   function expandBarHtml(file_name, opt_direction) {
     var html = '<div class="ExpandBar">' +
-        '<pre class="ExpandArea Expand' + ABOVE + '"></pre>' +
+        '<div class="ExpandArea Expand' + ABOVE + '"></div>' +
         '<div class="ExpandLinkContainer LinkContainer"><span class="ExpandText">expand: </span>';
 
     // FIXME: If there are <100 line to expand, don't show the expand-100 link.
@@ -367,7 +367,7 @@
         expandLinkHtml(BELOW, 100);
     }
 
-    html += '</div><pre class="ExpandArea Expand' + BELOW + '"></pre></div>';
+    html += '</div><div class="ExpandArea Expand' + BELOW + '"></div></div>';
     return html;
   }