<p>This is ready for another wave of reviewing, I think. It uses a layout and a partial now. Please do give feedback on the particulars of the implementation.</p>
<p>I've added "Hi, %{user}" at the top of the changeset comment notification mail. That particular mail is probably crucial for increasing new user engagement, and it didn't have a greeting line at all. I copied the line from the message notification locale strings, in every language. Is that OK, for me to create a new localized string like this, or is this going to confuse the translation pipeline (if there is one)?</p>
<p>As I mentioned above, there's also a wholly new string to be localized, it goes "To unsubscribe from updates to this changeset, visit ...".</p>
<p>We should probably discuss the visual appearance of the messages, too. I'm thinking of writing a diary entry to bring more community feedback into this.</p>

<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/1401#issuecomment-272572365">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLWgnajPOO17kUW9R-_VVACw1mU2Zks5rSAODgaJpZM4LYyok">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABWnLXTRQLfMhbCkJNKP93nJ_qUbNpvTks5rSAODgaJpZM4LYyok.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/openstreetmap/openstreetmap-website/pull/1401#issuecomment-272572365"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/openstreetmap/openstreetmap-website","title":"openstreetmap/openstreetmap-website","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/openstreetmap/openstreetmap-website"}},"updates":{"snippets":[{"icon":"PERSON","message":"@saintamh in #1401: This is ready for another wave of reviewing, I think. It uses a layout and a partial now. Please do give feedback on the particulars of the implementation.\r\n\r\nI've added \"Hi, %{user}\" at the top of the changeset comment notification mail. That particular mail is probably crucial for increasing new user engagement, and it didn't have a greeting line at all. I copied the line from the message notification locale strings, in every language. Is that OK, for me to create a new localized string like this, or is this going to confuse the translation pipeline (if there is one)?\r\n\r\nAs I mentioned above, there's also a wholly new string to be localized, it goes \"To unsubscribe from updates to this changeset, visit ...\".\r\n\r\nWe should probably discuss the visual appearance of the messages, too. I'm thinking of writing a diary entry to bring more community feedback into this."}],"action":{"name":"View Pull Request","url":"https://github.com/openstreetmap/openstreetmap-website/pull/1401#issuecomment-272572365"}}}</script>