{"action":"created","comment":{"url":"https://api.github.com/repos/ignatenkobrain/gnome-news/pulls/comments/35482897","id":35482897,"diff_hunk":"@@ -189,27 +189,43 @@ def __init__(self, tracker, post):\n               h1 {\n                 margin-bottom: 2px;\n               }\n+\n+              #footer {\n+                font-size: 14px;\n+                color: #777;\n+              }\n+\n+              #footer a:link,\n+              #footer a:active,\n+              #footer a:visited {\n+                text-decoration: none;\n+                color: #777;\n+              }\n+\n+              #footer a:hover {\n+                text-decoration: underline;\n+              }\n             </style>\n             <body>\n               <article>\n               <h1>%s</h1>\n-              <span>\n-            \"\"\" % post.title\n+              <span>%s</span>\n+              <p>%s</p>\n+              <div id=\"footer\">\"\"\" % (post.title, post.author, post.content)\n \n             if post.author_homepage:\n-                html += \"\"\"<a href=\"%s\">%s</a>\"\"\" % (post.author_homepage, post.author)\n-            else:\n-                html += \"\"\"%s\"\"\" % post.author\n-\n-            html += \"\"\"<p>%s</p>\"\"\" % post.content\n+                html += \"\"\"<p>%s</p>\"\"\" % post.author_homepage\n \n             if post.author_email:\n-                html += \"\"\"<small>%s</small>\"\"\" % post.author_email\n+                html += \"\"\"<p>%s</p>\"\"\" % post.author_email","path":"gnomenews/view.py","position":40,"original_position":40,"commit_id":"61f65ffaa3239d064e4e0fefd30ce132ee10b9d8","original_commit_id":"61f65ffaa3239d064e4e0fefd30ce132ee10b9d8","user":{"login":"felipeborges","id":365434,"avatar_url":"https://avatars.githubusercontent.com/u/365434?v=3","gravatar_id":"","url":"https://api.github.com/users/felipeborges","html_url":"https://github.com/felipeborges","followers_url":"https://api.github.com/users/felipeborges/followers","following_url":"https://api.github.com/users/felipeborges/following{/other_user}","gists_url":"https://api.github.com/users/felipeborges/gists{/gist_id}","starred_url":"https://api.github.com/users/felipeborges/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/felipeborges/subscriptions","organizations_url":"https://api.github.com/users/felipeborges/orgs","repos_url":"https://api.github.com/users/felipeborges/repos","events_url":"https://api.github.com/users/felipeborges/events{/privacy}","received_events_url":"https://api.github.com/users/felipeborges/received_events","type":"User","site_admin":false},"body":"I agree on the clickable email address. Speaking of the info position,\nthat's how it's in the new mockups.\nEm 24/07/2015 18:26, \"Vadim Rutkovsky\" <notifications@github.com> escreveu:\n\n> In gnomenews/view.py\n> <https://github.com/ignatenkobrain/gnome-news/pull/62#discussion_r35467525>\n> :\n>\n> >\n> >              if post.author_email:\n> > -                html += \"\"\"<small>%s</small>\"\"\" % post.author_email\n> > +                html += \"\"\"<p>%s</p>\"\"\" % post.author_email\n>\n> I think it would be best to combine author name with email and convert it\n> into a link.\n>\n> Not sure if having a post link below the post is a good idea - I'd put all\n> the metadata right after title\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/ignatenkobrain/gnome-news/pull/62/files#r35467525>.\n>\n","created_at":"2015-07-25T13:02:22Z","updated_at":"2015-07-25T13:02:22Z","html_url":"https://github.com/ignatenkobrain/gnome-news/pull/62#discussion_r35482897","pull_request_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/pulls/62","_links":{"self":{"href":"https://api.github.com/repos/ignatenkobrain/gnome-news/pulls/comments/35482897"},"html":{"href":"https://github.com/ignatenkobrain/gnome-news/pull/62#discussion_r35482897"},"pull_request":{"href":"https://api.github.com/repos/ignatenkobrain/gnome-news/pulls/62"}}},"pull_request":{"url":"https://api.github.com/repos/ignatenkobrain/gnome-news/pulls/62","id":40793717,"html_url":"https://github.com/ignatenkobrain/gnome-news/pull/62","diff_url":"https://github.com/ignatenkobrain/gnome-news/pull/62.diff","patch_url":"https://github.com/ignatenkobrain/gnome-news/pull/62.patch","issue_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/issues/62","number":62,"state":"open","locked":false,"title":"Each message should have a link to a full story/source","user":{"login":"felipeborges","id":365434,"avatar_url":"https://avatars.githubusercontent.com/u/365434?v=3","gravatar_id":"","url":"https://api.github.com/users/felipeborges","html_url":"https://github.com/felipeborges","followers_url":"https://api.github.com/users/felipeborges/followers","following_url":"https://api.github.com/users/felipeborges/following{/other_user}","gists_url":"https://api.github.com/users/felipeborges/gists{/gist_id}","starred_url":"https://api.github.com/users/felipeborges/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/felipeborges/subscriptions","organizations_url":"https://api.github.com/users/felipeborges/orgs","repos_url":"https://api.github.com/users/felipeborges/repos","events_url":"https://api.github.com/users/felipeborges/events{/privacy}","received_events_url":"https://api.github.com/users/felipeborges/received_events","type":"User","site_admin":false},"body":"Fixes #52","created_at":"2015-07-24T16:47:07Z","updated_at":"2015-07-25T13:02:22Z","closed_at":null,"merged_at":null,"merge_commit_sha":"9b6023ea93156e18f3c2743acee0aae20bba4a13","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/pulls/62/commits","review_comments_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/pulls/62/comments","review_comment_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/pulls/comments{/number}","comments_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/issues/62/comments","statuses_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/statuses/61f65ffaa3239d064e4e0fefd30ce132ee10b9d8","head":{"label":"ignatenkobrain:wip/felipe/post-footer","ref":"wip/felipe/post-footer","sha":"61f65ffaa3239d064e4e0fefd30ce132ee10b9d8","user":{"login":"ignatenkobrain","id":2866862,"avatar_url":"https://avatars.githubusercontent.com/u/2866862?v=3","gravatar_id":"","url":"https://api.github.com/users/ignatenkobrain","html_url":"https://github.com/ignatenkobrain","followers_url":"https://api.github.com/users/ignatenkobrain/followers","following_url":"https://api.github.com/users/ignatenkobrain/following{/other_user}","gists_url":"https://api.github.com/users/ignatenkobrain/gists{/gist_id}","starred_url":"https://api.github.com/users/ignatenkobrain/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ignatenkobrain/subscriptions","organizations_url":"https://api.github.com/users/ignatenkobrain/orgs","repos_url":"https://api.github.com/users/ignatenkobrain/repos","events_url":"https://api.github.com/users/ignatenkobrain/events{/privacy}","received_events_url":"https://api.github.com/users/ignatenkobrain/received_events","type":"User","site_admin":false},"repo":{"id":39015111,"name":"gnome-news","full_name":"ignatenkobrain/gnome-news","owner":{"login":"ignatenkobrain","id":2866862,"avatar_url":"https://avatars.githubusercontent.com/u/2866862?v=3","gravatar_id":"","url":"https://api.github.com/users/ignatenkobrain","html_url":"https://github.com/ignatenkobrain","followers_url":"https://api.github.com/users/ignatenkobrain/followers","following_url":"https://api.github.com/users/ignatenkobrain/following{/other_user}","gists_url":"https://api.github.com/users/ignatenkobrain/gists{/gist_id}","starred_url":"https://api.github.com/users/ignatenkobrain/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ignatenkobrain/subscriptions","organizations_url":"https://api.github.com/users/ignatenkobrain/orgs","repos_url":"https://api.github.com/users/ignatenkobrain/repos","events_url":"https://api.github.com/users/ignatenkobrain/events{/privacy}","received_events_url":"https://api.github.com/users/ignatenkobrain/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/ignatenkobrain/gnome-news","description":"","fork":false,"url":"https://api.github.com/repos/ignatenkobrain/gnome-news","forks_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/forks","keys_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/keys{/key_id}","collaborators_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/teams","hooks_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/hooks","issue_events_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/issues/events{/number}","events_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/events","assignees_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/assignees{/user}","branches_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/branches{/branch}","tags_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/tags","blobs_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/git/refs{/sha}","trees_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/git/trees{/sha}","statuses_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/statuses/{sha}","languages_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/languages","stargazers_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/stargazers","contributors_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/contributors","subscribers_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/subscribers","subscription_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/subscription","commits_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/commits{/sha}","git_commits_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/git/commits{/sha}","comments_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/comments{/number}","issue_comment_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/issues/comments{/number}","contents_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/contents/{+path}","compare_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/compare/{base}...{head}","merges_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/merges","archive_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/downloads","issues_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/issues{/number}","pulls_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/pulls{/number}","milestones_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/milestones{/number}","notifications_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/labels{/name}","releases_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/releases{/id}","created_at":"2015-07-13T13:55:09Z","updated_at":"2015-07-25T09:04:18Z","pushed_at":"2015-07-24T22:56:46Z","git_url":"git://github.com/ignatenkobrain/gnome-news.git","ssh_url":"git@github.com:ignatenkobrain/gnome-news.git","clone_url":"https://github.com/ignatenkobrain/gnome-news.git","svn_url":"https://github.com/ignatenkobrain/gnome-news","homepage":null,"size":837,"stargazers_count":13,"watchers_count":13,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":5,"mirror_url":null,"open_issues_count":33,"forks":5,"open_issues":33,"watchers":13,"default_branch":"master"}},"base":{"label":"ignatenkobrain:master","ref":"master","sha":"2e5568c5fd66b9b9eab42b9e6d5ed879666a1e0e","user":{"login":"ignatenkobrain","id":2866862,"avatar_url":"https://avatars.githubusercontent.com/u/2866862?v=3","gravatar_id":"","url":"https://api.github.com/users/ignatenkobrain","html_url":"https://github.com/ignatenkobrain","followers_url":"https://api.github.com/users/ignatenkobrain/followers","following_url":"https://api.github.com/users/ignatenkobrain/following{/other_user}","gists_url":"https://api.github.com/users/ignatenkobrain/gists{/gist_id}","starred_url":"https://api.github.com/users/ignatenkobrain/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ignatenkobrain/subscriptions","organizations_url":"https://api.github.com/users/ignatenkobrain/orgs","repos_url":"https://api.github.com/users/ignatenkobrain/repos","events_url":"https://api.github.com/users/ignatenkobrain/events{/privacy}","received_events_url":"https://api.github.com/users/ignatenkobrain/received_events","type":"User","site_admin":false},"repo":{"id":39015111,"name":"gnome-news","full_name":"ignatenkobrain/gnome-news","owner":{"login":"ignatenkobrain","id":2866862,"avatar_url":"https://avatars.githubusercontent.com/u/2866862?v=3","gravatar_id":"","url":"https://api.github.com/users/ignatenkobrain","html_url":"https://github.com/ignatenkobrain","followers_url":"https://api.github.com/users/ignatenkobrain/followers","following_url":"https://api.github.com/users/ignatenkobrain/following{/other_user}","gists_url":"https://api.github.com/users/ignatenkobrain/gists{/gist_id}","starred_url":"https://api.github.com/users/ignatenkobrain/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ignatenkobrain/subscriptions","organizations_url":"https://api.github.com/users/ignatenkobrain/orgs","repos_url":"https://api.github.com/users/ignatenkobrain/repos","events_url":"https://api.github.com/users/ignatenkobrain/events{/privacy}","received_events_url":"https://api.github.com/users/ignatenkobrain/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/ignatenkobrain/gnome-news","description":"","fork":false,"url":"https://api.github.com/repos/ignatenkobrain/gnome-news","forks_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/forks","keys_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/keys{/key_id}","collaborators_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/teams","hooks_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/hooks","issue_events_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/issues/events{/number}","events_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/events","assignees_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/assignees{/user}","branches_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/branches{/branch}","tags_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/tags","blobs_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/git/refs{/sha}","trees_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/git/trees{/sha}","statuses_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/statuses/{sha}","languages_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/languages","stargazers_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/stargazers","contributors_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/contributors","subscribers_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/subscribers","subscription_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/subscription","commits_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/commits{/sha}","git_commits_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/git/commits{/sha}","comments_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/comments{/number}","issue_comment_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/issues/comments{/number}","contents_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/contents/{+path}","compare_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/compare/{base}...{head}","merges_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/merges","archive_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/downloads","issues_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/issues{/number}","pulls_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/pulls{/number}","milestones_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/milestones{/number}","notifications_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/labels{/name}","releases_url":"https://api.github.com/repos/ignatenkobrain/gnome-news/releases{/id}","created_at":"2015-07-13T13:55:09Z","updated_at":"2015-07-25T09:04:18Z","pushed_at":"2015-07-24T22:56:46Z","git_url":"git://github.com/ignatenkobrain/gnome-news.git","ssh_url":"git@github.com:ignatenkobrain/gnome-news.git","clone_url":"https://github.com/ignatenkobrain/gnome-news.git","svn_url":"https://github.com/ignatenkobrain/gnome-news","homepage":null,"size":837,"stargazers_count":13,"watchers_count":13,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":5,"mirror_url":null,"open_issues_count":33,"forks":5,"open_issues":33,"watchers":13,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/ignatenkobrain/gnome-news/pulls/62"},"html":{"href":"https://github.com/ignatenkobrain/gnome-news/pull/62"},"issue":{"href":"https://api.github.com/repos/ignatenkobrain/gnome-news/issues/62"},"comments":{"href":"https://api.github.com/repos/ignatenkobrain/gnome-news/issues/62/comments"},"review_comments":{"href":"https://api.github.com/repos/ignatenkobrain/gnome-news/pulls/62/comments"},"review_comment":{"href":"https://api.github.com/repos/ignatenkobrain/gnome-news/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/ignatenkobrain/gnome-news/pulls/62/commits"},"statuses":{"href":"https://api.github.com/repos/ignatenkobrain/gnome-news/statuses/61f65ffaa3239d064e4e0fefd30ce132ee10b9d8"}}}}