<p></p>
<p><b>@gravitystorm</b> commented on this pull request.</p>

<p>This seems like a straightforward PR, but unfortunately it opens a can of worms.</p>
<p>As <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/mmd-osm/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/mmd-osm">@mmd-osm</a> notes, this proposal breaks the diary list pages, which uses the same partial. I'm not sure what the best solution is here, since I'm happy to move from links to buttons to denote 'actions' (and trying to edit a diary entry seems like an 'action' to me). Perhaps one solution is to use two different partials, and make the diary list a "summary" of each entry (e.g. with a max word limit) rather than including the entire entry. Or alternatively, perhaps we keep the partial just for the summaries, and move the "full display" directly into the view.</p>
<p>Beyond that, one more minor point:</p>
<ul>
<li>The need for pt-3 and mt-3 is not clear. I think the pt-3 is because the location partial is "bare", i.e. text not wrapped in a paragraph, and so I would look into changing the html first, to avoid needing any explicit padding/margins.</li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/3019#pullrequestreview-559112711">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLO23PBSDTRSWZ4JWPTSXBWFPANCNFSM4VK2SZGQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AAK2OLMDCLAZK3MA3N3Z5ODSXBWFPA5CNFSM4VK2SZG2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOEFJWEBY.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/3019#pullrequestreview-559112711",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/3019#pullrequestreview-559112711",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>