Skip to content

Conversation

@domhhv
Copy link
Owner

@domhhv domhhv commented Jan 27, 2026

Closes #251

Summary by CodeRabbit

New Features

  • Added "Log occurrence" button to the calendar week view for tracking occurrences more easily
  • Redesigned note interaction with a dedicated button UI for improved usability
  • Enhanced calendar layout with refined spacing and better organization of day headers

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 27, 2026

📝 Walkthrough

Walkthrough

The WeekCalendar component is enhanced with a new "Log occurrence" button for each day, powered by the useOccurrenceDrawerActions hook. The existing note UI is refactored to use a button-based component, and layout styling is adjusted to accommodate both action buttons with improved spacing and visual hierarchy.

Changes

Cohort / File(s) Summary
UI and Hook Integration
src/components/calendar/WeekCalendar.tsx
Added CalendarBlankIcon import and useOccurrenceDrawerActions hook. Refactored note UI from icon-only to button-based control. Introduced new "Log occurrence" button wired to openOccurrenceDrawer({ dayToLog: day }). Both buttons grouped in a container with shared and separate tooltips.
Layout and Styling
src/components/calendar/WeekCalendar.tsx
Made header container sticky-left. Expanded day cell width to min-w-32. Adjusted padding/margins (px-8 py-4) and removed larger lg-specific padding. Updated grid container padding alignment.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐰 A button here, a button there—
Week view's getting its logging flair!
Sticky headers and calendar days,
Habits logged in splendid ways! 📋✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 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 habit occurrence logging UI to the week calendar view.
Linked Issues check ✅ Passed The changes implement all requirements from #251: a UI button near each day that opens OccurrenceDialog for logging habits.
Out of Scope Changes check ✅ Passed All changes are directly related to adding habit occurrence logging to the week calendar; layout adjustments support the new functionality.
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 27, 2026

#170 Bundle Size — 1.82MiB (+0.02%).

225ab22(current) vs cc57d4f main#169(baseline)

Warning

Bundle contains 2 duplicate packages – View duplicate packages

Bundle metrics  Change 2 changes Regression 1 regression
                 Current
#170
     Baseline
#169
Regression  Initial JS 1.08MiB(+0.04%) 1.08MiB
No change  Initial CSS 0B 0B
Change  Cache Invalidation 100% 87.09%
No change  Chunks 8 8
No change  Assets 9 9
No change  Modules 6673 6673
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
#170
     Baseline
#169
Regression  JS 1.59MiB (+0.02%) 1.59MiB
Regression  CSS 241.08KiB (+0.02%) 241.04KiB

Bundle analysis reportBranch feat/week-calendar-habit-loggingProject dashboard


Generated by RelativeCIDocumentationReport issue

@domhhv domhhv merged commit 8e3fefa into main Jan 27, 2026
11 checks passed
@domhhv domhhv deleted the feat/week-calendar-habit-logging branch January 27, 2026 18:21
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.

Add habit logging to calendar week view

2 participants