Improve calendar time gutter sticky positioning and alignment#692
Improve calendar time gutter sticky positioning and alignment#6922witstudios wants to merge 1 commit intomasterfrom
Conversation
- Use top-0 -translate-y-1/2 instead of -top-2.5 to properly center time labels on hour grid lines - Skip the 12 AM label (hour 0) which was clipped above the scroll container boundary due to negative positioning - Make time gutter sticky with background so it stays visible and doesn't get obscured during horizontal scroll - Add leading-none to prevent extra line-height from affecting alignment https://claude.ai/code/session_012VnPGU2n6WUM4Z856xxgDU
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
No actionable comments were generated in the recent review. 🎉 📝 WalkthroughWalkthroughBoth DayView and WeekView calendar components receive styling and rendering updates: the left time gutter becomes sticky with proper layering, hourly labels render conditionally starting from hour 1, and vertical centering is adjusted via CSS transforms. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
Summary
Enhanced the time gutter in both DayView and WeekView calendar components to be sticky during horizontal scrolling and improved the visual alignment of hour labels.
Key Changes
sticky left-0 bg-background z-10) so it remains visible when scrolling horizontally through calendar events-top-2.5totop-0 -translate-y-1/2for more precise vertical centeringleading-noneto prevent line-height from affecting alignmenthour > 0conditional to reduce visual clutterImplementation Details
The time gutter now uses CSS
stickypositioning with proper z-index layering to ensure it stays visible during horizontal scrolling while maintaining the calendar's visual hierarchy. The hour label centering uses CSS transforms for more reliable alignment across different browsers and font sizes.https://claude.ai/code/session_012VnPGU2n6WUM4Z856xxgDU
Summary by CodeRabbit