<p></p>
<p>Thanks for the PR, I reviewed it yesterday. This proposal is similar to <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="565308089" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/2537" data-hovercard-type="pull_request" data-hovercard-url="/openstreetmap/openstreetmap-website/pull/2537/hovercard" href="https://github.com/openstreetmap/openstreetmap-website/pull/2537">#2537</a>, and it has similar problems to <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="565308089" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/2537" data-hovercard-type="pull_request" data-hovercard-url="/openstreetmap/openstreetmap-website/pull/2537/hovercard" href="https://github.com/openstreetmap/openstreetmap-website/pull/2537">#2537</a> , namely</p>
<ul>
<li>It introduces more custom CSS classes, which we're trying to remove</li>
<li>It only addresses the welcome message, and not the other similar situations like the sidebar panels.</li>
</ul>
<p>Based on some of the discussions on <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="565308089" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/2537" data-hovercard-type="pull_request" data-hovercard-url="/openstreetmap/openstreetmap-website/pull/2537/hovercard" href="https://github.com/openstreetmap/openstreetmap-website/pull/2537">#2537</a> by <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/bezdna/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/bezdna">@bezdna</a> , I've started work on replacing the h2+close button by a flex-box based solution. This in particular addresses the point made about the close button not being related to the header itself (and so it shouldn't appear within the h2 element). I ran out of time this week before finishing it, but you can see what I'm thinking at <a class="commit-link" data-hovercard-type="commit" data-hovercard-url="https://github.com/gravitystorm/openstreetmap-website/commit/b3fc5bc71a4fd7a5b13aee3cad1c527de244da4e/hovercard" href="https://github.com/gravitystorm/openstreetmap-website/commit/b3fc5bc71a4fd7a5b13aee3cad1c527de244da4e">gravitystorm@<tt>b3fc5bc</tt></a> - feedback is welcome. I intend to use the same solution (maybe not the exact same partial, but at least the same divs and flexbox layout) for all of our current close buttons, so that they behave consistently.</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/2872#issuecomment-709367093">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLOCSEV7PP5D55ZQ6E3SK4B3NANCNFSM4SA2IIXA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AAK2OLKWKKOSSSE5C34X55LSK4B3NA5CNFSM4SA2IIXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFJEBKNI.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/2872#issuecomment-709367093",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/2872#issuecomment-709367093",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>