How would you like me to address this issue? Adam
On Thu, Apr 22, 2010 at 1:01 PM, Darin Adler <[email protected]> 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 > [email protected] > http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev > _______________________________________________ webkit-dev mailing list [email protected] http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
