<p></p>
<p><b>@AntonKhorev</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5609#discussion_r1942930666">app/views/notes/show.html.erb</a>:</p>
<pre style='color:#555'>> @@ -52,10 +52,10 @@
     <% end %>
   </div>
 
-  <% if @note_comments.length > 1 %>
+  <% if @note_comments.length > (!@note.author.nil? && @note.author.status == "deleted" ? 0 : 1) %>
</pre>
<p dir="auto">I think it's better to make <code class="notranslate">@note_comments</code> contain true comments in the controller instead of having this condition checked twice here.</p>
<p dir="auto"><code class="notranslate">@note_comments</code> were used in three places of this view: description, anonymous warning and discussion.</p>
<ul dir="auto">
<li>Description is handled differently now.</li>
<li>Discussion is going to work with all <code class="notranslate">@note_comments</code> without deciding which to drop.</li>
<li>Anonymous warning will need to check if either note author is nil or any comment's author is nil. Maybe this check is also better done in the controller and saved to some variable.</li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5609#pullrequestreview-2595757498">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLOCI5A4AZZW3FHDWD32OIFRDAVCNFSM6AAAAABWQX7BVWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKOJVG42TONBZHA">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLO627NKY6XPCQPJWX32OIFRDA5CNFSM6AAAAABWQX7BVWWGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTU2XAO3U.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><openstreetmap/openstreetmap-website/pull/5609/review/2595757498</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/5609#pullrequestreview-2595757498",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/5609#pullrequestreview-2595757498",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>