<p></p>
<p><b>@nertc</b> commented on this pull request.</p>

<p dir="auto">Other than this one comment, I have no more questions. Functionality, routing and highlighting work as intended.</p>
<p dir="auto">On big screens edit button seems a bit lonely because of the big gap and visual difference with other menu items. I tried adding paddings and changing colors of it, but neither of them improved visual sufficiently. It's out of scope of this PR, but, in the future, it will be great if we redesign menu to make it more aligned with the new arrangement of buttons.</p>
<p dir="auto">Also, changing <code class="notranslate">Export</code> text with the export icon, may end up gaining even more space and a cleaner visual without many menu items.</p>
<p dir="auto">Overall, this PR fixes the issue in a clean, functional way.</p><hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5151#discussion_r1758524990">app/assets/javascripts/index.js</a>:</p>
<pre style='color:#555'>> @@ -412,6 +412,9 @@ $(document).ready(function () {
 
     if (OSM.router.route(this.pathname + this.search + this.hash)) {
       e.preventDefault();
+      if (this.pathname !== "/directions") {
</pre>
<p dir="auto">Why do we need explicit check for the directions path? If I understand correctly, this line of code is for keeping menu opened when navigating through directions pages, but app\assets\javascripts\index\directions.js:123:5 still adds "close" class to the header when it gets routes.</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/5151#pullrequestreview-2302621396">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLPMAPW63KRRYWQNZW3ZWK6JHAVCNFSM6AAAAABNRB7VROVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGMBSGYZDCMZZGY">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLMZYRWSS66DH44A4X3ZWK6JHA5CNFSM6AAAAABNRB7VROWGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTUJH43NI.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/5151/review/2302621396</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/5151#pullrequestreview-2302621396",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/5151#pullrequestreview-2302621396",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>