On Sun, Jun 12, 2011 at 1:30 PM, Adam Barth <[email protected]> wrote:

> The following text appears next to every patch in bugs.webkit.org:
>
>    Review Patch | Details | Formatted Diff | Diff
>
> Does anyone click the "Diff" link?  I clicked by accident the other
> day and my eyes were scared.  Should we remove it?
>

I use Diff/Formatted Diff to extract some lines out of a patch.  I can't do
that on review tool because I can't select lines by a mouse drag.

Similarly, the Review Patch page has these buttons on the bottom:
>
>    Bug 62XXX (contributor) r: ? cq: ? [Preview] [Publish]
>
> Does anyone click the "Preview" button?  If not, we should probably remove
> it.
>

I frequently use review to reduce the amount of typos and other forms of
errors before submission.

- Ryosuke
_______________________________________________
webkit-dev mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev

Reply via email to