<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/4158490?s=20&v=4" /><strong>AntonKhorev</strong> left a comment <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5981#issuecomment-2853129199">(openstreetmap/openstreetmap-website#5981)</a></div>
<blockquote>
<p dir="auto">Is there a specific reason you do this in pixel coordinates instead of lat/lon coordinates where you can use <code class="notranslate">changeset.bounds.contains(map.getBounds())</code>?</p>
</blockquote>
<p dir="auto">In pixels I take bbox border widths into account, which is maybe not a big difference.</p>
<p dir="auto">I can use <code class="notranslate">.contains</code> (and <code class="notranslate">.overlaps</code>) in pixel coordinates too, which I'm going to do for now. Those manual comparisons that I'm removing may come back if I'll do another feature to indicate where bboxes are located relative to the map view.</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/5981#issuecomment-2853129199">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLNQNFQ7N6ULOMV773325APQFAVCNFSM6AAAAAB4OKCNGWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQNJTGEZDSMJZHE">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLOG4CG4H3WITMU35Z325APQFA5CNFSM6AAAAAB4OKCNGWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTVKB5F66.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/5981/c2853129199</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/5981#issuecomment-2853129199",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/5981#issuecomment-2853129199",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>