The popup bubble is cute, but I think we can do without the titlebar. It doesn't really add anything (we already know it's Review Board) and it takes up space.

Can you use a <label> and appropriate style so it's bold and black for the labels?

A few things that stand out immediately.

1) Make sure to only use spaces, not tabs, for indentation. Much of the new code is using tabs, and it stands out in the diff viewer.

2) The diff wasn't generated correctly and doesn't apply to a few of the files. How was the diff generated? Make sure it's a branch off of an up-to-date 'master' and you're using post-review.

Can you upload a new screenshot with the new hover?

Sorry for the late review on this. You'll have it today.

Two blank lines between these.

Should be combined:

margin: 18px auto 0 auto;

Seems this should be alongside the ".review .reviewer" rule in reviews.css.

Two blank lines between these.

All this logic appears to be duplicated. It should be consolidated. You should be able to do:

$("#submitter, .reviewer a").hover(...)

I think this can be:

$(this).append(
    $("<div .../>")
    .load($(this).attr(...)));

No space before the "("

This should include the local_site checks, like the other views.

Space after the ":" in the dictionary.

Should have one space indentation for each nested tag.

Also, can you use "user-infobox" instead of "preview" for the IDs? "preview" is too generic and could potentially be used in other places.

I don't think we need the <br/>.

Instead of <b>..</b> and the <br/>, do:

<p><label>{% trans "Name:" %}</label> {{whatever}}</p>

You may need to tweak the styles for the margins on the <p> tags in this case. Maybe for <label> too, depending on which color you get (I imagine it'll be brown).

The rest of the templates specifically specify a format string. This should use:

{{user.last_login|date:"F jS, Y, P"}}

Same with date_joined.

No blank lines between these.

I imagine we should be able to use datagrid.html as the base?

Pretty sure this isn't the title you want.

Shouldn't reload.

If we use datagrid.html, we should get these for free.

Indentation is off on most things. Should always be consistent and use one space.

Each of these should use <label> instead of <div class=".."> and localize the text using {% trans %}

Shouldn't use <b> for the values, either.

Also, make sure to use the values recommended above for the infobox.

Use {% trans %} to localize this.

Space after {%

This needs to do the same local_site_name URL matching and logic as the other URLs, so that users can't grab info on users from other sites.

Committed to master (4dd548286edf1bee390e7775984890145ada8385) along with some modifications to the hover box view to make it play nice with the new LocalSite stuff and to show the requested user instead of always request.user (we both missed this the first time).