Change week view to Monday-Sunday calendar weeks and default to week … #147
Conversation
…view - Changed default graph view from 'day' to 'week' - Updated week calculation to use calendar weeks (Monday-Sunday) instead of rolling 7-day window - Fixed duplicate Sunday issue by filtering chart data to only include days within range - Updated both current and previous week calculations for consistency - Enabled Tide Goals feature in dev mode
|
I am missing a few things: -Interactivity to the week graph bars so clicking on a day navigates to that day view,
|
| } | ||
|
|
||
| return Object.entries(buckets) | ||
| .sort(([keyA], [keyB]) => { |
There was a problem hiding this comment.
can you provide more information about what this change is about? I'm assuming there is a bug?
There was a problem hiding this comment.
The date keys (e.g., "2025-11-03", "2025-11-04") were coming out in random order. This caused the graph to show days like: Tue, Wed, Mon... etc(wrong order). Adding localeCompare() sorts the ISO date strings alphabetically, which gives us chronological order: Mon... Sun
There was a problem hiding this comment.
Are they not currently in order? I haven't seen this issue yet
…view