user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
* [PATCH] www: always show subject for root of thread skeleton
@ 2022-09-01  8:42 Eric Wong
  0 siblings, 0 replies; only message in thread
From: Eric Wong @ 2022-09-01  8:42 UTC (permalink / raw)
  To: meta

For users with short attention spans, the root message of should
have the Subject, since <title> is often truncated in most browsers.
  Just a minor aesthetic improvement I noticed while overhauling
  large parts of View*.pm to reduce memory traffic and use.
  And this does have a tiny memory reduction, too :P

 lib/PublicInbox/ | 1 -
 1 file changed, 1 deletion(-)

diff --git a/lib/PublicInbox/ b/lib/PublicInbox/
index f29fff50..2e5228e1 100644
--- a/lib/PublicInbox/
+++ b/lib/PublicInbox/
@@ -777,7 +777,6 @@ sub thread_skel ($$$) {
 	# when multiple Subject: headers are present, so we follow suit:
 	my $subj = $hdr->header('Subject') // '';
 	$subj = '(no subject)' if $subj eq '';
-	$ctx->{prev_subj} = [ split(/ /, subject_normalized($subj)) ];
 	$ctx->{cur} = $mid;
 	$ctx->{prev_attr} = '';
 	$ctx->{prev_level} = 0;

^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2022-09-01  8:42 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-01  8:42 [PATCH] www: always show subject for root of thread skeleton Eric Wong

Code repositories for project(s) associated with this public inbox

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).