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

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5426#discussion_r1899523876">app/assets/javascripts/leaflet.layers.js</a>:</p>
<pre style='color:#555'>> @@ -31,6 +31,9 @@ L.OSM.layers = function (options) {
         var miniMap = L.map(mapContainer[0], { attributionControl: false, zoomControl: false, keyboard: false })
           .addLayer(new layer.constructor(layer.options));
 
+        if (layer.options.schemeClass) miniMap.getPane("tilePane").classList.add(layer.options.schemeClass);
+        miniMap.getPane("tilePane").style.setProperty("--dark-mode-map-filter", layer.options.filter || "none");
</pre>
<p dir="auto">If the dark flag/class should have special treatment, should thunderforest-cycle get a special <code class="notranslate">nofilter</code> class as the first layer where the default from the current proposal <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="2661819843" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/5328" data-hovercard-type="issue" data-hovercard-url="/openstreetmap/openstreetmap-website/issues/5328/hovercard?comment_id=2539051340&comment_type=issue_comment" href="https://github.com/openstreetmap/openstreetmap-website/issues/5328#issuecomment-2539051340">#5328 (comment)</a> is explicitly asked for (<a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="2733542733" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/5395" data-hovercard-type="issue" data-hovercard-url="/openstreetmap/openstreetmap-website/issues/5395/hovercard?comment_id=2538451264&comment_type=issue_comment" href="https://github.com/openstreetmap/openstreetmap-website/issues/5395#issuecomment-2538451264">#5395 (comment)</a>)? And should further layers also get a class each?</p>
<p dir="auto">Do you think it is favorable to spread the map layer filter definitions that much into CSS as well? I think having the way the layers from leaflet-osm are used in a single location including filters adds to the maintainability.</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/5426#discussion_r1899523876">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLJP3P77UCWQJKAGYR32IE6URAVCNFSM6AAAAABT7CQ4MWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKMRVGU2TEMJYHE">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLO6HULKIE52BMNMKR32IE6URA5CNFSM6AAAAABT7CQ4MWWGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTUWRDPD2.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/5426/review/2525552189</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/5426#discussion_r1899523876",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/5426#discussion_r1899523876",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>