mARTin-B78/My-Weekly-ToDo-List
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: pair setWeekStartDay with saveSetting in SettingsSidebar

Browse Source 
Phase 2 migrated weekStartDay reads to profile.weekStartDay, exposing a
pre-existing gap: the Mon/Sun buttons called setWeekStartDay() without
saveSetting(), so profile.weekStartDay never updated on click.

v1.80.1
This commit is contained in:
mARTin 2026-03-30 23:06:12 +02:00
parent bfb749a8a6
commit 8a0af281a5
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/components/SettingsSidebar.tsx
View File

@ -1340,13 +1340,13 @@ function SettingsSidebar({
<div style={{ display: "flex", gap: "8px" }}> <div style={{ display: "flex", gap: "8px" }}>
<button <button
className={`px-3 py-2 rounded text-sm ${weekStartDay === 1 ? "bg-blue-600 text-white" : "bg-gray-100 text-gray-700 dark:bg-gray-700 dark:text-gray-300"}`} className={`px-3 py-2 rounded text-sm ${weekStartDay === 1 ? "bg-blue-600 text-white" : "bg-gray-100 text-gray-700 dark:bg-gray-700 dark:text-gray-300"}`}
onClick={() => setWeekStartDay(1)} onClick={() => { setWeekStartDay(1); saveSetting("weekStartDay", 1); }}
> >
{t.monday} {t.monday}
</button> </button>
<button <button
className={`px-3 py-2 rounded text-sm ${weekStartDay === 0 ? "bg-blue-600 text-white" : "bg-gray-100 text-gray-700 dark:bg-gray-700 dark:text-gray-300"}`} className={`px-3 py-2 rounded text-sm ${weekStartDay === 0 ? "bg-blue-600 text-white" : "bg-gray-100 text-gray-700 dark:bg-gray-700 dark:text-gray-300"}`}
onClick={() => setWeekStartDay(0)} onClick={() => { setWeekStartDay(0); saveSetting("weekStartDay", 0); }}
> >
{t.sunday} {t.sunday}
</button> </button>