view: conditionally anchor to thread skeleton

We only care about the thread skeleton if we have
multiple messages in a thread, single message threads
can just go to the top of the message.

1 files changed, 9 insertions, 2 deletions

diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm
index 3d6348c7..49068641 100644
--- a/lib/PublicInbox/View.pm
+++ b/lib/PublicInbox/View.pm
@@ -905,11 +905,18 @@ sub dump_topics {
 
                 # $n isn't the total number of posts on the topic,
                 # just the number of posts in the current results window
-                $n = $n == 1 ? '' : " ($n+ messages)";
+                my $anchor;
+                if ($n == 1) {
+                        $n = '';
+                        $anchor = '#u'; # top of only message
+                } else {
+                        $n = " ($n+ messages)";
+                        $anchor = '#t'; # thread skeleton
+                }
 
                 my $mbox = qq(<a\nhref="$href/t.mbox.gz">mbox.gz</a>);
                 my $atom = qq(<a\nhref="$href/t.atom">Atom</a>);
-                my $s = "<a\nhref=\"$href/T/#t\"><b>$top</b></a>\n" .
+                my $s = "<a\nhref=\"$href/T/$anchor\"><b>$top</b></a>\n" .
                         " $ts UTC $n - $mbox / $atom\n";
                 for (my $i = 0; $i < scalar(@ex); $i += 2) {
                         my $level = $ex[$i];