From 2b3233ce3b309279ee3df8b99a471aa2e8c435e3 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Sun, 30 Aug 2015 01:51:22 +0000 Subject: [PATCH] view: remove "threadlink" from thread view We're already inside the thread, and our thread summary inside /m/$MESSAGE_ID/ is already sufficient got navigate back to the /t/$MESSAGE_ID/ page. So I think it's sufficient to keep the /t/$MESSAGE_ID/ page lighter with fewer links and avoid introducing strange terminology. In contrast, "permalink" is relatively well-known and not an alien term to readers: https://en.wikipedia.org/wiki/Permalink --- lib/PublicInbox/View.pm | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm index 7cac7851..3ee67902 100644 --- a/lib/PublicInbox/View.pm +++ b/lib/PublicInbox/View.pm @@ -74,13 +74,10 @@ sub index_entry { my $path = $root_anchor ? '../../' : ''; my $href = $mid->as_href; my $irt = $header_obj->header('In-Reply-To'); - my ($anchor_idx, $anchor, $t_anchor); + my ($anchor_idx, $anchor); if (defined $irt) { $anchor_idx = anchor_for($irt); $anchor = $seen->{$anchor_idx}; - $t_anchor = T_ANCHOR; - } else { - $t_anchor = ''; } if ($srch) { $subj = "$subj"; @@ -134,11 +131,6 @@ sub index_entry { $rv .= " parent"; } - if ($srch) { - $rv .= " " . - "threadlink"; - } - $fh->write($rv .= ''); } -- 2.44.0