fix(weekly-view): correct time slot offset and clipped slots

[Akhilesh-Varute]

What does this PR do?

Fixes two layout bugs in the weekly calendar view.

• Fix SchedulerColumns marginTop fallback from var(--gridDefaultSize) (58px) to var(--calendar-offset-top) (28px) so the event grid aligns correctly with time labels on first render
• Fix falsy check for topOffsetMinutes === 0 in Cell and CustomCell so slots at the start hour are correctly positioned with absolute + top: 0

Fixes #14208

Visual Demo

Before: Time slots are offset from the time labels on the left, and end-hour slots get clipped (as shown in the issue screenshots).

After: Slots align correctly with time labels and no slots are clipped.

Mandatory Tasks (DO NOT REMOVE)

• I have self-reviewed the code (A decent size PR without self-review might be rejected).
• I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox. N/A
• I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

1. Go to any booker page (e.g. /alan)
2. Switch to Weekly view
3. Verify that time slots align correctly with the hour labels on the left
4. Verify that all slots within an hour are visible (not clipped)
5. Test with a timezone behind UTC (e.g. Pacific TZ -07) where the bug was most reproducible

No environment variables needed. A standard cal.com local setup is sufficient.

[CLAassistant]

All committers have signed the CLA.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9db94bf1-6f42-4c13-8bae-895eb41a8307

📥 Commits

Reviewing files that changed from the base of the PR and between 3c52f57 and 0ccf3d7.

📒 Files selected for processing (2)

• packages/features/calendars/weeklyview/components/event/Empty.tsx
• packages/features/calendars/weeklyview/components/grid/index.tsx

---

📝 Walkthrough

Walkthrough

The pull request updates CSS positioning logic in the calendar weekly view components. In the event component, truthiness checks for topOffsetMinutes are made explicit by using strict inequality (!== undefined) instead of loose truthy checks, allowing zero values to be valid offsets. In the grid component, the fallback CSS variable for marginTop is changed from var(--gridDefaultSize) to var(--calendar-offset-top) when offsetHeight is falsy. These changes refine how positioning and margin calculations are handled in calendar rendering.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the two main fixes: correcting time slot offset and clipped slots in weekly view, matching the core changes.
Description check	✅ Passed	The description is directly related to the changeset, detailing the two layout bug fixes, visual demo, testing instructions, and referencing the linked issue.
Linked Issues check	✅ Passed	The PR directly addresses issue #14208 by fixing both reported problems: misaligned time slots and clipped end-hour slots through CSS positioning corrections.
Out of Scope Changes check	✅ Passed	All changes are scoped to fixing the two layout bugs in weekly view components (Empty.tsx and index.tsx) with no unrelated modifications.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Akhilesh-Varute]

Hi, could a maintainer please add the run-ci label so CI checks can run? Thanks!

[romitg2]

please add before/after image/video demo

[Akhilesh-Varute]

Before: 12:00am slot showing at 1:00am row (offset bug)

After: 1:00am slot perfectly aligned with 1:00am label