about summary refs log tree commit homepage
path: root/lib
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2016-07-21 01:23:02 +0000
committerEric Wong <e@80x24.org>2016-07-21 01:23:18 +0000
commit803914775816f7de5485fa397bad64e1bd404381 (patch)
treea73ce57ebdbd843b8450a9b64861effb54b47ec5 /lib
parent8888a03c12dcd0f17ff611f4617523942a3aed49 (diff)
downloadpublic-inbox-803914775816f7de5485fa397bad64e1bd404381.tar.gz
Clearly label "Thread overview" and "Reply instructions"
so users can quickly skip stuff they're not interested in.

Additionally, note the fact the thread view allows quick
navigation within the thread to avoid extra network requests
and improve the display for single-message threads.

Finally, use <hr> to better-delineate sections of each page.
Diffstat (limited to 'lib')
-rw-r--r--lib/PublicInbox/View.pm19
-rw-r--r--lib/PublicInbox/WwwStream.pm2
2 files changed, 13 insertions, 8 deletions
diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm
index 49068641..5f99644f 100644
--- a/lib/PublicInbox/View.pm
+++ b/lib/PublicInbox/View.pm
@@ -61,10 +61,12 @@ sub msg_reply {
         push @$arg, '/path/to/YOUR_REPLY';
         $arg = join(" \\\n    ", '', @$arg);
         <<EOF
-<pre
-id=R>You may reply publically to <a
-href=#t>this message</a> via
-plain-text email using any one of the following methods:
+<hr><pre
+id=R><b>Reply instructions:</b>
+
+You may reply publically to <a
+href=#t>this message</a> via plain-text email
+using any one of the following methods:
 
 * Save the following mbox file, import it into your mail client,
   and reply-to-all from there: <a
@@ -326,10 +328,12 @@ sub thread_html {
         return missing_thread($ctx) if $nr == 0;
         my $skel = '<hr><pre>';
         $skel .= $nr == 1 ? 'only message in thread' : 'end of thread';
-        $skel .= ", back to <a\nhref=\"../../\">index</a>";
-        $skel .= "\n<a\nid=t>$nr+ messages in thread:</a> (download: ";
-        $skel .= "<a\nhref=\"../t.mbox.gz\">mbox.gz</a>";
+        $skel .= ", back to <a\nhref=\"../../\">index</a>\n\n";
+        $skel .= "<b\nid=t>Thread overview:</b> ";
+        $skel .= $nr == 1 ? '(only message)' : "$nr+ messages";
+        $skel .= " (download: <a\nhref=\"../t.mbox.gz\">mbox.gz</a>";
         $skel .= " / follow: <a\nhref=\"../t.atom\">Atom feed</a>)\n";
+        $skel .= "-- links below jump to the message on this page --\n";
         $ctx->{-upfx} = '../../';
         $ctx->{cur_level} = 0;
         $ctx->{dst} = \$skel;
@@ -523,6 +527,7 @@ sub thread_skel {
                         qq(/ <a\nhref="${tpfx}t.atom">Atom feed</a>);
 
         my $parent = in_reply_to($hdr);
+        $$dst .= "\n<b>Thread overview: </b>";
         if ($nr <= 1) {
                 if (defined $parent) {
                         $$dst .= "($expand)\n ";
diff --git a/lib/PublicInbox/WwwStream.pm b/lib/PublicInbox/WwwStream.pm
index bce384b7..bdc8048d 100644
--- a/lib/PublicInbox/WwwStream.pm
+++ b/lib/PublicInbox/WwwStream.pm
@@ -80,7 +80,7 @@ sub _html_end {
                         join("\n", map { "\tgit clone --mirror $_" } @urls);
         }
         my $url = PublicInbox::Hval::prurl($ctx->{env}, PI_URL);
-        '<pre>'.join("\n",
+        '<hr><pre>'.join("\n",
                 '- ' . $desc,
                 $urls,
                 'Served with public-inbox: '.