Skip to content

Conversation

@domhhv
Copy link
Owner

@domhhv domhhv commented Jan 29, 2026

Summary by CodeRabbit

  • New Features
    • Added a real-time current-time indicator to the week calendar that displays the current hour position when viewing today's schedule. The indicator updates every minute to accurately reflect the current time within the hourly grid.

✏️ Tip: You can customize this high-level summary in your review settings.

@domhhv domhhv self-assigned this Jan 29, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 29, 2026

📝 Walkthrough

Walkthrough

Added a useCurrentTime hook that updates every 60 seconds to track the real-time now value. WeekCalendar integrates this hook to render a current-time indicator div in the hourly grid, positioned vertically based on current minutes, appearing only when the displayed day is today.

Changes

Cohort / File(s) Summary
Real-time Current-Time Indicator
src/components/calendar/WeekCalendar.tsx
Introduced useCurrentTime hook with 60-second interval refresh mechanism. Added conditional rendering of a current-time marker in hourly grid cells when day is today, with vertical position calculated from current minutes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 A tick-tock on the calendar grid,
Each minute marked, each second bid,
When today arrives with gleaming light,
The rabbit's time-marker shines so bright! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding a visual marker for the current hour in the week calendar view.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
  • 📝 Generate docstrings

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

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

@relativeci
Copy link

relativeci bot commented Jan 29, 2026

#184 Bundle Size — 1.83MiB (+0.02%).

26ea2f5(current) vs 33000e0 main#183(baseline)

Warning

Bundle contains 2 duplicate packages – View duplicate packages

Bundle metrics  Change 2 changes Regression 1 regression
                 Current
#184
     Baseline
#183
Regression  Initial JS 1.08MiB(+0.03%) 1.08MiB
No change  Initial CSS 0B 0B
Change  Cache Invalidation 100% 0%
No change  Chunks 8 8
No change  Assets 9 9
No change  Modules 6672 6672
No change  Duplicate Modules 0 0
No change  Duplicate Code 0% 0%
No change  Packages 223 223
No change  Duplicate Packages 2 2
Bundle size by type  Change 2 changes Regression 2 regressions
                 Current
#184
     Baseline
#183
Regression  JS 1.59MiB (+0.02%) 1.59MiB
Regression  CSS 242.22KiB (+0.02%) 242.18KiB

Bundle analysis reportBranch feat/today-line-in-week-calendarProject dashboard


Generated by RelativeCIDocumentationReport issue

@domhhv domhhv merged commit 0900e8f into main Jan 29, 2026
11 checks passed
@domhhv domhhv deleted the feat/today-line-in-week-calendar branch January 29, 2026 19:40
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