diff options
| author | Eric Wong <e@80x24.org> | 2017-03-24 01:41:11 +0000 |
|---|---|---|
| committer | Eric Wong <e@80x24.org> | 2017-03-24 01:46:04 +0000 |
| commit | 4ba430497bfb4763691b0ad8af573eb6ff420b96 (patch) | |
| tree | 828ccdf64dfbb8f2f97f933fda832afba54e500d /lib/PublicInbox/SearchView.pm | |
| parent | b24abb7762a600a3cba45ccd9bcb5ec8d97dd37d (diff) | |
| download | public-inbox-4ba430497bfb4763691b0ad8af573eb6ff420b96.tar.gz | |
searchview: show full (&x=t) messages in ascending chronlogical order
When displaying search results with full messages, it makes more sense to show them in ascending chronological order when going by date. Reverse chronological order makes more sense for search results which only show the subject.
Diffstat (limited to 'lib/PublicInbox/SearchView.pm')
| -rw-r--r-- | lib/PublicInbox/SearchView.pm | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/PublicInbox/SearchView.pm b/lib/PublicInbox/SearchView.pm index b867d002..cec87c6a 100644 --- a/lib/PublicInbox/SearchView.pm +++ b/lib/PublicInbox/SearchView.pm @@ -185,9 +185,9 @@ sub mset_thread { $pct{$smsg->mid} = $i->get_percent; $smsg; } ($mset->items) ]}); - + my $r = $q->{r}; my $rootset = PublicInbox::SearchThread::thread($msgs, - $q->{r} ? sort_relevance(\%pct) : *PublicInbox::View::sort_ts); + $r ? sort_relevance(\%pct) : *PublicInbox::View::sort_ts); my $skel = search_nav_bot($mset, $q). "<pre>"; my $inbox = $ctx->{-inbox}; $ctx->{-upfx} = ''; @@ -203,11 +203,11 @@ sub mset_thread { PublicInbox::View::walk_thread($rootset, $ctx, *PublicInbox::View::pre_thread); - + @$msgs = reverse @$msgs if $r; my $mime; sub { return unless $msgs; - while ($mime = shift @$msgs) { + while ($mime = pop @$msgs) { $mime = $inbox->msg_by_smsg($mime) and last; } if ($mime) { |