Add review UIs to the diff viewer.

Review Request #4479 — Created Aug. 22, 2013 and submitted — Latest diff uploaded

Information

Review Board
master

Reviewers

Add review UIs to the diff viewer.

We now display review UIs in the diff viewer for file attachments that
have an inline-capable review UI. This takes the place of the thumbnail
that we would otherwise render.

When displaying an inline-capable review UI, the "Review" button goes
away, and the review UI is just rendered inline. It can be commented on,
just like normal.

At the moment, both the original and modified versions each have a
review UI capable of commenting. It's unclear whether the left-hand side
should allow commenting on a review UI, but right now, it's there. Extra
work would need to be done to tell the AbstractReviewables that they're
not allowed to add commenting capabilities.

The next step is to add diffable file attachment support to review UIs.
Tested with some image files with the ImageReviewUI.allow_inline = True.
I was able to comment on the images. Reloading the page didn't cache the
old comment data, so I saw my new comments. They appeared in the review.

Tested with allow_inline = False, and it went to the previous behavior of
having a Review link, and just showing thumbnails.

    Loading...