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

<p>Thanks for the PR. I'm happy to see more styling being added to tables within diary entries and other places where we have our "richtext" content.</p>
<p>However, we need to change the approach used. We've moved to Bootstrap for most of our styling, and so we should not be adding more custom styling rules to common.scss any longer. Bootstrap gives us various table styling options (<a href="https://getbootstrap.com/docs/4.5/content/tables/" rel="nofollow">https://getbootstrap.com/docs/4.5/content/tables/</a>) out of the box, and so we should use one of them.</p>
<p>Since bootstrap tables are opt-in, i.e. require a class to be added to the <code><table></code> tag, then that will need to be implemented as part of the RichText.to_html process (in <a href="https://github.com/openstreetmap/openstreetmap-website/blob/4e78f3ae5b0d96f599840e181c1a4be4335dded9/lib/rich_text.rb"><code>lib/rich_text.rb</code></a>)</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/2959#pullrequestreview-533597182">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLNUCDPX4NCVFFD4YHLSQPVMHANCNFSM4TVFPHOA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AAK2OLOJQZ2ZJNR6GICQ2JDSQPVMHA5CNFSM4TVFPHOKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOD7HAX7Q.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/2959#pullrequestreview-533597182",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/2959#pullrequestreview-533597182",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>