One of the other demos put together at the hackathon let you drag on the patch to select how much context you wanted when making your comment. I can try to make that work.
Adam On Thu, Apr 22, 2010 at 1:09 PM, Adam Barth <aba...@webkit.org> wrote: > How would you like me to address this issue? > > Adam > > > On Thu, Apr 22, 2010 at 1:01 PM, Darin Adler <da...@apple.com> wrote: >> I’m having trouble reviewing patches with the action=review patch since the >> experimental commenting support was added. I now have to do a lot of editing >> and copying and pasting when reviewing that was not necessary before. >> >> - The action=review JavaScript code now deletes the copy of the patch, so I >> can't cite things by hand. Before I would never need to copy and paste when >> reviewing, just delete things, but now if I do want to cite more than one >> line I need to open another window. >> >> - The comments all cite only a single line of the patch and I almost never >> have a comment that's for a single line. So I have to do a lot of editing, >> pretending to comment on multiple lines. >> >> (The action=review JavaScript page still includes an entire copy of the >> patch, and then the script code deletes it. Inelegant, and easy to fix!) >> >> -- Darin >> >> _______________________________________________ >> webkit-dev mailing list >> webkit-dev@lists.webkit.org >> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev >> > _______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
