<div style="display: flex; flex-wrap: wrap; white-space: pre-wrap; align-items: center; "><img height="20" width="20" style="border-radius:50%; margin-right: 4px;" decoding="async" src="https://avatars.githubusercontent.com/u/147741?s=20&v=4" /><strong>tomhughes</strong> left a comment <a href="https://github.com/openstreetmap/openstreetmap-website/pull/6302#issuecomment-3165133850">(openstreetmap/openstreetmap-website#6302)</a></div>
<p dir="auto">Speaking as the review of <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="3279898292" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/6271" data-hovercard-type="pull_request" data-hovercard-url="/openstreetmap/openstreetmap-website/pull/6271/hovercard" href="https://github.com/openstreetmap/openstreetmap-website/pull/6271">#6271</a> the declared purpose of that PR was moving styling from CSS to bootstrap classes rather than any optimisation of the CSS itself.</p>
<p dir="auto">It was also essentially impossible to do a detailed code review of it, or it would have taken an unreasonable amount of time for me to go and read up on what each bootstrap class that was added did and correlate that to removed CSS and vice versa.</p>
<p dir="auto">So review was largely based on a visual inspection of the site with the PR merged to check that there were no obvious issues in the areas of the site affected by the changes, so mostly I was concerned with the header and the search bar and so on. I probably only looked at a normal browser and not a mobile size though. Also I had probably already cancelled any banners on my browser so wouldn't have seen them again.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/6302#issuecomment-3165133850">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLPXSL5W7HX6MAKZIKL3MOESLAVCNFSM6AAAAACDLKFGECVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTCNRVGEZTGOBVGA">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLI44TX73LRZ5MMAC4T3MOESLA5CNFSM6AAAAACDLKFGECWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTV4VAOBU.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><openstreetmap/openstreetmap-website/pull/6302/c3165133850</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>

<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/6302#issuecomment-3165133850",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/6302#issuecomment-3165133850",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>