about summary refs log tree commit homepage
path: root/lib
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2015-08-20 02:43:20 +0000
committerEric Wong <e@80x24.org>2015-08-20 02:44:27 +0000
commit69ed3f6087171f460e7aa4398b1eaac690a53033 (patch)
treecbabff7fd3999fa94b76e3fc7c6a6dc3d419e886 /lib
parent574026d755f5f0c27817d0b60ed7119669b2951b (diff)
downloadpublic-inbox-69ed3f6087171f460e7aa4398b1eaac690a53033.tar.gz
Add some spacing between topics to improve readability when
scanning or in case a subject gets too long.

The title and Atom feed may not be highly-visible otherwise.
While we're at it, use the proper "Atom feed" terminology since
some folks may not understand just what "atom" means.
Diffstat (limited to 'lib')
-rw-r--r--lib/PublicInbox/Feed.pm7
1 files changed, 4 insertions, 3 deletions
diff --git a/lib/PublicInbox/Feed.pm b/lib/PublicInbox/Feed.pm
index 40dfb451..2e352cba 100644
--- a/lib/PublicInbox/Feed.pm
+++ b/lib/PublicInbox/Feed.pm
@@ -64,7 +64,8 @@ sub generate_html_index {
         my $html = "<html><head><title>$title</title>" .
                 "<link\nrel=alternate\ntitle=\"Atom feed\"\n".
                 "href=\"$atom_url\"\"\ntype=\"application/atom+xml\"/>" .
-                '</head><body>' . PublicInbox::View::PRE_WRAP;
+                '</head><body>' . PublicInbox::View::PRE_WRAP .
+                "<b>$title</b> (<a\nhref=\"$atom_url\">Atom feed</a>)\n";
 
         my $state;
         my $git = PublicInbox::GitCatFile->new($ctx->{git_dir});
@@ -329,14 +330,14 @@ sub add_topic {
 sub dump_topics {
         my ($dst, $topics) = @_;
         my ($order, $subjs) = @$topics;
-        $$dst .= '[No recent topics]' unless (scalar @$order);
+        $$dst .= "\n[No recent topics]" unless (scalar @$order);
         while (defined(my $info = shift @$order)) {
                 my ($mid, $ts, $u, $subj) = @$info;
                 my $n = delete $subjs->{$subj};
                 $mid = PublicInbox::Hval->new($mid)->as_href;
                 $subj = PublicInbox::Hval->new($subj)->as_html;
                 $u = PublicInbox::Hval->new($u)->as_html;
-                $$dst .= "<a\nhref=\"t/$mid.html#u\"><b>$subj</b></a>\n- ";
+                $$dst .= "\n<a\nhref=\"t/$mid.html#u\"><b>$subj</b></a>\n- ";
                 $ts = POSIX::strftime('%Y-%m-%d %H:%M', gmtime($ts));
                 if ($n == 1) {
                         $$dst .= "created by $u @ $ts UTC\n"