<p>I've done a pass through all the code now (haven't looked at tests yet) and put comments in line for your consideration.</p>

<p>In terms of UI issues there are a small number of things where I'd like to here what people think:</p>

<ul class="task-list">
<li>Subscribe button location - not sure I like it where it is but don't have better ideas at the moment.</li>
<li>Styling of the "login to comment" button.</li>
</ul><p>There's also a major issue with the hide/unhide button - most people won't be able to see that but it looks horrible where it is. I suspect we need to make it a lot more subtle - maybe just a simple hide/unhide link at the end of the comment attribution line?</p>

<p>The other critical thing before we can merge is making sure everybody is happy with the API changes, which basically means:</p>

<ul class="task-list">
<li>Adding the <code>include_discussion</code> flag to the changeset API call.</li>
<li>The related changes to the XML for a changeset to include the comments.</li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/openstreetmap/openstreetmap-website/pull/772#issuecomment-50163749">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/1419053__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcyMTkyMDcxNSwiZGF0YSI6eyJpZCI6MzU4NjU3NjJ9fQ==--bf7f4fcc55e1c0229d1931fa20701152a8ac0fa1.gif" width="1" /></p>