[webkit-dev] Review tool changes

[Oliver Hunt]

I really would like to be able to select some text and add a comment that uses the selection as context, a single line of context is frequently insufficient, this is about the only thing that still makes the new review tool less effective than the old review mechanism (for me at least).

--Oliver

On Sep 20, 2010, at 11:32 AM, Darin Adler wrote:

> On Sep 20, 2010, at 10:22 AM, Darin Fisher wrote:
> 
>> How about this?
>> 
>> If any annotations were made to the patch, then "the button" gets named Preview.  Else, the button is named "Publish" and when clicked performs its work in one shot.
>> 
>> Was there a strong outcry for removing the preview step?  I only found it bothersome when I wanted to issue a quick r=me on a patch that didn't require any additional changes.
> 
> Good idea. Here’s another idea:
> 
> The button starts out named Preview and works with a preview.
> 
> If the review or commit-queue flag is altered then the button is changed to “Publish” and works without a preview.
> 
> Thus if I like the preview work flow, I don’t set those flags until I get to the preview page. If I like the faster work flow, I can set a flag and then push Publish.
> 
> One type of person who is not served by my proposal is someone who wants to add comments only and not set a flag but wants the faster work flow.
> 
>    -- Darin
> 
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev