Changeset 39111c0 in Klonkt

Timestamp:

08/06/2026 02:27:26 AM (5 weeks ago)

Author:

Claude (agent) <aiclaude@…>

Branches:

main

Children:

971d00b

Parents:

50489c7

git-author:

Robin <roboburr@…> (08/06/2026 02:27:25 AM)

git-committer:

Claude (agent) <aiclaude@…> (08/06/2026 02:27:26 AM)

Message:

Binnen een draad staat de tegenpartij nog maar een keer

Robins melding: de tegenpartij stond er drie keer -- in de uitsparing, en dan
per bubbel nog eens als "You -> @handle". Daaronder herhaalde elke bubbel ook
nog "replied via the fediverse", terwijl dat voor de hele draad geldt. Samen
kostte dat twee regels per bericht aan ruimte die de inhoud toekomt.

Binnen een draad valt nu weg: de handle achter de naam, en de standaardzin over
wat voor soort bericht het is. De naam zelf blijft, want je moet zien WIE wat
zei. Buiten een draad verandert er niets: daar dragen die twee wel informatie.

Wat blijft staan omdat het geen boilerplate is: de hulpvraag-boei, de zwaai, het
slotje bij een prive-bericht en de View-original-links. Is er daarna niets
betekenisvols over, dan vervalt de hele regel in plaats van als lege div ruimte
te blijven pakken.

Gecontroleerd op de dev-data: van de twee bubbels houdt alleen die met een
View-original-link nog een msg-what-regel, en er staat geen herhaalde handle
meer in. Suite 410/410.

Co-Authored-By: Claude Opus 5 <noreply@…>

File:

• src/views/partials/msg-item.ejs (modified) (4 diffs)

src/views/partials/msg-item.ejs

@@ -77 +77 @@
               <% if (_t === 'sent') { %>
                 <span class="msg-who"><%= t('msg.you') %></span>
-                <% if (n.to_handle) { %><span class="msg-handle">→ <%= n.to_handle %></span><% } %>
+                <% if (n.to_handle && !_sub) { %><span class="msg-handle">→ <%= n.to_handle %></span><% } %>
               <% } else { %>
                 <a class="msg-who" href="<%= n.url %>" target="_blank" rel="nofollow noopener"><%- emojiName(_who, n.actor_emoji_json) %></a>
-                <% if (n.handle && n.name) { %><span class="msg-handle"><%= n.handle %></span><% } %>
+                <% /* De naam blijft: binnen een draad moet je zien WIE wat zei.
+                      De handle erachter niet -- die staat in de kop. */ %>
+                <% if (n.handle && n.name && !_sub) { %><span class="msg-handle"><%= n.handle %></span><% } %>
               <% } %>
               <% if (_new) { %><span class="msg-new" title="<%= t('msg.new') %>"></span><% } %>
@@ -89 +91 @@
             </div>
 
+            <% var _hasWhat = !_sub || _priv
+                 || (_t === 'mention' && (n.help_request || n.wave || n.note_url))
+                 || (_t === 'sent' && n.in_reply_to); %>
+            <% if (_hasWhat) { %>
             <div class="msg-what">
               <% if (n.count && n.count > 1) { %>
@@ -99 +105 @@
               <% } else if (_t === 'mention' && n.help_request) { %>🛟 <%= t('msg.help_request') %>
               <% } else if (_t === 'mention' && n.wave) { %>👋 <%= t('msg.waved_at_you') %>
+              <% } else if (_sub) { %><%# binnen een draad zegt de kop al dat dit een gesprek is %>
               <% } else if (_t === 'mention') { %><%= t('notif.mentioned') %>
               <% } else if (_t === 'sent') { %><%= t('msg.sent_reply') %>
@@ -108 +115 @@
               <% if (_t === 'sent' && n.in_reply_to) { %><a class="msg-post" href="<%= n.in_reply_to %>" target="_blank" rel="nofollow noopener"><%= t('tl.view_original') %></a><% } %>
             </div>
+            <% } %>
 
             <% if (_t === 'report' && n.content) { %><div class="msg-content"><%= n.content %></div>