<p>In app/controllers/browse_controller.rb:</p>
<pre style='color:#555'>> @@ -58,6 +58,7 @@ def node_history
>    def changeset
>      @type = "changeset"
>      @changeset = Changeset.find(params[:id])
> +    @comments = @changeset.comments.unscope(:where => :visible).includes(:author)
</pre>
<p>I think we should only remove the visibility restriction if the user is a moderator, or if we don't do that we should only render them in the view if the user is a moderator.</p>

<p>The class based client side show/hide stuff that you're using at the moment was only ever intended to be used where we couldn't (for caching reasons) do the restriction on the server, and that doesn't apply here (or anywhere really now as we have pretty much given up on caching).</p>

<p>On top of that, there may well be comments that we have to hide and which we really don't want showing up in the source at all even if the javascript does make them invisible.</p>

<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/files#r15401286">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/1419053__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcyMTkxNTY2NiwiZGF0YSI6eyJpZCI6MzU4NjU3NjJ9fQ==--6e4055d40ac1e41fe5ffe5483b25557669a06efc.gif" width="1" /></p>