jeisinger

e.g. on https://codereview.chromium.org/476833003/#msg6 you first have to click on "7 older comments", and then comment 6 is highlighted. It should just expand immediately.

e.g. links like this https://codereview.chromium.org/422593003/diff/500001/include/v8.h#newcode4359 (from the comments) drop you out of the review tool into rietveld. I'd like the diff/patchset of that file to expand and scroll up instead.

when you're not logged in, and you try to send a message, you get "error: undefined"

2

It shouldn't let you write a message in the first place I think or have a better error message

Diff display should show comments outside the diff range

1

e.g. in https://codereview.chromium.org/393123005/ when you click on show diffs for patchset 1, the comment in line 35 is not shown

show context link displayed on diffs without base files

2

e.g. in https://codereview.chromium.org/351863002/ for generated_resources.grd since there is no base file, clicking the show context link does nothing. Either disable the link, or magically pull context from svn :)