I frequently copy &amp; paste function names, variable names, etc... into my comment so I like the new behavior.<div><br></div><div>- Ryosuke<br><br><div class="gmail_quote">On Wed, Nov 10, 2010 at 10:18 AM, Adam Barth <span dir="ltr">&lt;<a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a>&gt;</span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Yeah, Darin and Alexey asked for this change.  I thought about<br>
emailing webkit-dev, but I was worried that would have been too spamy.<br>
 Hopefully the new behavior works for everyone.<br>
<br>
Adam<br>
<div><div></div><div><br>
<br>
On Wed, Nov 10, 2010 at 6:36 AM, David Levin &lt;<a href="mailto:levin@chromium.org" target="_blank">levin@chromium.org</a>&gt; wrote:<br>
&gt; (Single) Click on the line numbers to leave comments. Not the lines in the<br>
&gt; code review.<br>
&gt; I&#39;m just pointing this out because I only just noticed and thought it was<br>
&gt; broken when I couldn&#39;t double click lines. (There is text at the top of the<br>
&gt; page but I claim to have banner blindness.)<br>
&gt; dave<br>
&gt;<br>
&gt; ---------- Forwarded message ----------<br>
&gt; From: &lt;<a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a>&gt;<br>
&gt; Date: Mon, Nov 8, 2010 at 2:18 PM<br>
&gt; Subject: [webkit-changes] [71570] trunk/BugsSite<br>
&gt; To: <a href="mailto:webkit-changes@lists.webkit.org" target="_blank">webkit-changes@lists.webkit.org</a><br>
&gt;<br>
&gt;<br>
&gt; Revision 71570 Author <a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a> Date 2010-11-08 14:18:05 -0800 (Mon,<br>
&gt; 08 Nov 2010)<br>
&gt;<br>
&gt; Log Message<br>
&gt;<br>
&gt; 2010-11-08  Adam Barth  &lt;<a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a>&gt;<br>
&gt;<br>
&gt;         Reviewed by Eric Seidel.<br>
&gt;<br>
&gt;         Restore normal meaning of double-click on patch review page?<br>
&gt;         <a href="https://bugs.webkit.org/show_bug.cgi?id=47641" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=47641</a><br>
&gt;<br>
&gt;         After this patch, you need to click on the line numbers to add a<br>
&gt;         comment.<br>
&gt;<br>
&gt;         * PrettyPatch/PrettyPatch.rb:<br>
&gt;         * code-review.js:<br>
&gt;<br>
&gt; Modified Paths<br>
&gt;<br>
&gt; trunk/BugsSite/ChangeLog<br>
&gt; trunk/BugsSite/PrettyPatch/PrettyPatch.rb<br>
&gt; trunk/BugsSite/code-review.js<br>
&gt;<br>
&gt; Diff<br>
&gt;<br>
&gt; Modified: trunk/BugsSite/ChangeLog (71569 =&gt; 71570)<br>
&gt;<br>
&gt; --- trunk/BugsSite/ChangeLog  2010-11-08 22:10:51 UTC (rev 71569)<br>
&gt; +++ trunk/BugsSite/ChangeLog  2010-11-08 22:18:05 UTC (rev 71570)<br>
&gt; @@ -1,3 +1,16 @@<br>
&gt; +2010-11-08  Adam Barth  &lt;<a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a>&gt;<br>
&gt; +<br>
&gt; +        Reviewed by Eric Seidel.<br>
&gt; +<br>
&gt; +        Restore normal meaning of double-click on patch review page?<br>
&gt; +        <a href="https://bugs.webkit.org/show_bug.cgi?id=47641" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=47641</a><br>
&gt; +<br>
&gt; +        After this patch, you need to click on the line numbers to add a<br>
&gt; +        comment.<br>
&gt; +<br>
&gt; +        * PrettyPatch/PrettyPatch.rb:<br>
&gt; +        * code-review.js:<br>
&gt; +<br>
&gt;  2010-10-09  Adam Barth  &lt;<a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a>&gt;<br>
&gt;<br>
&gt;          Reviewed by Alexey Proskuryakov.<br>
&gt;<br>
&gt; Modified: trunk/BugsSite/PrettyPatch/PrettyPatch.rb (71569 =&gt; 71570)<br>
&gt;<br>
&gt; --- trunk/BugsSite/PrettyPatch/PrettyPatch.rb 2010-11-08 22:10:51 UTC (rev<br>
&gt; 71569)<br>
&gt; +++ trunk/BugsSite/PrettyPatch/PrettyPatch.rb 2010-11-08 22:18:05 UTC (rev<br>
&gt; 71570)<br>
&gt; @@ -312,7 +312,7 @@<br>
&gt;  }<br>
&gt;  &lt;/style&gt;<br>
&gt;  &lt;script<br>
&gt; src=&quot;<a href="https://ajax.googleapis.com/ajax/libs/jquery/1.4.2/jquery.min.js" target="_blank">https://ajax.googleapis.com/ajax/libs/jquery/1.4.2/jquery.min.js</a>&quot;&gt;&lt;/script&gt;<br>
&gt; -&lt;script src=&quot;code-review.js?version=13&quot;&gt;&lt;/script&gt;<br>
&gt; +&lt;script src=&quot;code-review.js?version=14&quot;&gt;&lt;/script&gt;<br>
&gt;  EOF<br>
&gt;<br>
&gt;      def self.revisionOrDescription(string)<br>
&gt;<br>
&gt; Modified: trunk/BugsSite/code-review.js (71569 =&gt; 71570)<br>
&gt;<br>
&gt; --- trunk/BugsSite/code-review.js     2010-11-08 22:10:51 UTC (rev 71569)<br>
&gt; +++ trunk/BugsSite/code-review.js     2010-11-08 22:18:05 UTC (rev 71570)<br>
&gt; @@ -253,7 +253,7 @@<br>
&gt;    }<br>
&gt;<br>
&gt;    function crawlDiff() {<br>
&gt; -    $(&#39;.Line&#39;).each(idify).each(hoverify).dblclick(addCommentField);<br>
&gt; +    $(&#39;.Line&#39;).each(idify).each(hoverify);<br>
&gt;      $(&#39;.FileDiff&#39;).each(function() {<br>
&gt;        var file_name = $(this).children(&#39;h1&#39;).text();<br>
&gt;        files[file_name] = this;<br>
&gt;<br>
&gt; _______________________________________________<br>
&gt; webkit-changes mailing list<br>
&gt; <a href="mailto:webkit-changes@lists.webkit.org" target="_blank">webkit-changes@lists.webkit.org</a><br>
&gt; <a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes</a><br>
&gt;<br>
&gt;<br>
&gt;<br>
</div></div>&gt; _______________________________________________<br>
&gt; webkit-dev mailing list<br>
&gt; <a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
&gt; <a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
&gt;<br>
&gt;<br>
_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
</blockquote></div><br></div>