From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-4.0 required=3.0 tests=ALL_TRUSTED,BAYES_00 shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id E32CE1F5AE; Wed, 28 Apr 2021 06:55:22 +0000 (UTC) From: Eric Wong To: meta@public-inbox.org Cc: Son Luong Ngoc Subject: [PATCH] view: add [thread overview] anchor next to Date: Date: Wed, 28 Apr 2021 06:55:22 +0000 Message-Id: <20210428065522.12795-1-e@80x24.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit List-Id: The existing Subject: anchor to #r may not be 100% obvious, and we can't stick the phrase "[thread overview]" into the same line as the Subject without introducing ambiguity. Fortunately, we have the Date: header directly under it. Adding "[thread overview]" after the Date: is unambiguous and won't make the line too long for valid emails. This hopefully improves navigation ever-so-slightly thanks to comments by Son Luong Ngoc. Reported-by: Son Luong Ngoc Link: https://public-inbox.org/git/YHhfsqfTJ9NzRwS1@C02YX140LVDN.corpad.adbkng.com/ --- lib/PublicInbox/View.pm | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm index f4f6da11..530c878f 100644 --- a/lib/PublicInbox/View.pm +++ b/lib/PublicInbox/View.pm @@ -694,7 +694,7 @@ sub _msg_page_prepare_obuf { for my $v ($eml->header('Date')) { $v = ascii_html($v); obfuscate_addrs($obfs_ibx, $v) if $obfs_ibx; # possible :P - $rv .= "Date: $v\n"; + $rv .= qq{Date: $v\t[thread overview]\n}; } if (!$nr) { # first (and only) message, common case $ctx->{-title_html} = join(' - ', @title); @@ -739,7 +739,8 @@ sub thread_skel ($$$) { $$skel .= SKEL_EXPAND."\n "; $$skel .= ghost_parent('../', $parent) . "\n"; } else { - $$skel .= '[no followups] '.SKEL_EXPAND."\n"; + $$skel .= "[no followups] ". + SKEL_EXPAND."\n"; } $ctx->{next_msg} = undef; $ctx->{parent_msg} = $parent;