2011-01-12 Ojan Vafai <ojan@chromium.org>
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 13 Jan 2011 00:20:07 +0000 (00:20 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 13 Jan 2011 00:20:07 +0000 (00:20 +0000)
        Reviewed by Mihai Parparita.

        fix assorted bugs with expansion lines
        https://bugs.webkit.org/show_bug.cgi?id=52326

        -Clicking "all" would collapse whitespace in the expanded lines.
        -Converted the remaining line creation code to using the DOM.

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

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

Websites/bugs.webkit.org/ChangeLog
Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
Websites/bugs.webkit.org/code-review.js

index 91789dd..33ed7db 100644 (file)
@@ -1,5 +1,18 @@
 2011-01-12  Ojan Vafai  <ojan@chromium.org>
 
+        Reviewed by Mihai Parparita.
+
+        fix assorted bugs with expansion lines
+        https://bugs.webkit.org/show_bug.cgi?id=52326
+
+        -Clicking "all" would collapse whitespace in the expanded lines.
+        -Converted the remaining line creation code to using the DOM.
+
+        * PrettyPatch/PrettyPatch.rb:
+        * code-review.js:
+
+2011-01-12  Ojan Vafai  <ojan@chromium.org>
+
         Reviewed by Adam Barth.
 
         review tool formatted diff doesn't match the uploaded diff
index 6c8cb3c..c73eb97 100644 (file)
@@ -317,7 +317,7 @@ body {
   border-right-color: #69F;
 }
 
-.ExpandArea {
+.ExpandArea, .ExpandLinkContainer {
   margin: 0;
   white-space: pre-wrap;
 }
index 696a08a..81ef64e 100644 (file)
     for (var i = 0; i < end_line_num - start_line_num; i++) {
       // FIXME: from line numbers are wrong
       var line_number = start_from_line_num + i + 1;
-      var contents = patched_file_contents[file_name][start_line_num + i];
+      var contents = $('<span class="text"></span>');
+      contents.text(patched_file_contents[file_name][start_line_num + i]);
       var line = is_side_by_side ? sideBySideExpansionLine(line_number, contents) : unifiedExpansionLine(line_number, contents);
       fragment.appendChild(line[0]);
     }
 
   function convertExpansionLine(diff_type, line) {
     var convert_function = diff_type == 'sidebyside' ? sideBySideExpansionLine : unifiedExpansionLine;
-    var contents = textContentsFor(line);
+    var contents = $('.text', line);
     var line_number = fromLineNumber(line);
     var new_line = convert_function(line_number, contents);
     $(line).replaceWith(new_line);