Skip to content

Fix Inconsistent Starting Date in Calendar UI#2159

Closed
KhoaNguyen706 wants to merge 2 commits into
devfrom
khoa/fixStartDate
Closed

Fix Inconsistent Starting Date in Calendar UI#2159
KhoaNguyen706 wants to merge 2 commits into
devfrom
khoa/fixStartDate

Conversation

@KhoaNguyen706
Copy link
Copy Markdown
Collaborator

closes #2158

On first load of the current month, we find which week row contains today in the grid and only render from that week down so dates like May 31 aren’t clipped off the bottom. The “Today” button does the same; changing month with the arrows or dropdown still shows the full month.

Before:
image

After fixing:
image

@KhoaNguyen706 KhoaNguyen706 changed the title Khoa/fix start date Fix Inconsistent Starting Date in Calendar UI Jun 1, 2026
Copy link
Copy Markdown
Contributor

@adarshm11 adarshm11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

my issue might have been overkill...

i think it makes more sense if we just manually enforce that the ui always shows the entire month, which could be up to 5 weeks. lets change the css to accomplish that, then we can create a chained pr to address any other issues.

update the screenshots for this new fix when you have them , and we can iterate from there

@adarshm11
Copy link
Copy Markdown
Contributor

ACTUALLY i have inspiration, after you finish the stuff i asked for in the prev comment, lets make ui updates so we can view the calendar in day, week, month, or year view (like how google calendar does) BUT do this IN A SEPARATE PR that is branched off of this branch :)

@KhoaNguyen706
Copy link
Copy Markdown
Collaborator Author

my issue might have been overkill...

i think it makes more sense if we just manually enforce that the ui always shows the entire month, which could be up to 5 weeks. lets change the css to accomplish that, then we can create a chained pr to address mobile ui.

update the screenshots for this new fix when you have them , and we can iterate from there

ACTUALLY i have inspiration, after you finish the stuff i asked for in the prev comment, lets make ui updates so we can view the calendar in day, week, month, or year view (like how google calendar does) BUT do this IN A SEPARATE PR that is branched off of this branch :)

Okay I got it, i have to fix that first and then Mobile Ui and another pr. Will do later. thanks

@adarshm11
Copy link
Copy Markdown
Contributor

actually i think the issue is small enough to where we can address later, i will rewrite issue and close this pr for now

@adarshm11 adarshm11 closed this Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants