<p></p>
<blockquote>
<p dir="auto">Oh, I forgot about this. But the user can now set his "Preferred Website Color Scheme" which should be the basis of the theme not the browser setting.</p>
</blockquote>
<p dir="auto">Thank you for the suggestion. I’ll refactor accordingly to ensure alignment with the preferred color scheme setting.</p>
<blockquote>
<p dir="auto">It is a very minor nitpick, but is it possible to dynamically determine what's the first weekday based on the user's locale and adjust the heatmap accordingly? <a href="https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/Locale/getWeekInfo" rel="nofollow">Intl.Locale.prototype.getWeekInfo()</a> could be used. It isn't currently available in Firefox but all other browsers have it.</p>
<p dir="auto">P.S. Added <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1937867" rel="nofollow">bug 1937867</a> to Firefox's bug tracker.</p>
</blockquote>
<p dir="auto">I tried dynamically adjusting the start and end weekdays using <a href="https://cal-heatmap.com/docs/options/subDomain#ghday" rel="nofollow">ghDay</a> as the subdomain, but it rounds to the first and last weeks of the month, making implementation tricky. Accommodating this would require significant effort, which may not be worth it for such a minor adjustment.</p>
<blockquote>
<p dir="auto">What's the source(s) of all the javascript in <code class="notranslate">vendor/</code>? I would expect to see <code class="notranslate">Vendorfile</code> updated, or preferably if these are sourced from e.g. node modules, then they should be included in package.json and added using the assets pipeline. If any of these are source files, i.e. written just for this project, then they should be in app/.</p>
</blockquote>
<p dir="auto">To be honest I wasn’t  too familiar with how our assets pipeline works,so I just added the <code class="notranslate">cal-heatmap</code> library and its dependencies in <code class="notranslate">vendor/</code> and added required code in <code class="notranslate">manifest.js</code> and <code class="notranslate">application.js</code> to make it load correctly. If using <code class="notranslate">package.json</code> is preferred, I can update the PR accordingly. I'm not sure if there is some additional config needed for this to be accomplished or would updating the <code class="notranslate">package.json</code> file would be enough.</p>
<p dir="auto">Thank you for pointing out the existing <code class="notranslate">popper</code> inclusion. I’ll remove the duplicate entry from <code class="notranslate">application.js</code> and the <code class="notranslate">vendor</code> folder.</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/5402#issuecomment-2552523845">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLMUBC7KM52E2L7E5XD2GIHH7AVCNFSM6AAAAABTWNRZO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJSGUZDGOBUGU">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLOUSAKFXDQB6ZPHCRL2GIHH7A5CNFSM6AAAAABTWNRZO6WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUYERWEK.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/5402/c2552523845</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/5402#issuecomment-2552523845",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/5402#issuecomment-2552523845",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>