refactor: split CalendarView controller from rendering #367

New issue

Closed

opened 2026-05-11 21:02:07 +00:00 by barrettruth · 0 comments

barrettruth commented

2026-05-11 21:02:07 +00:00

Owner

Parent: #355

Problem

CalendarView still owns view state, FullCalendar sync, optimistic recurring-instance state, drag/drop mutation, popover state, navigation, keybindings, and rendering in one large component.

Solution

Split behavior-preserving controller logic into focused hooks/components:

calendar navigation/view-mode state
keyboard handling
optimistic event/materialized recurrence state
keep FcCalendar and visible calendar behavior unchanged

Acceptance criteria

CalendarView becomes a smaller composition layer.
Week/day/month navigation, drag/drop, recurrence edits, and popovers keep working.
Focused tests cover extracted pure/controller logic where practical.

Parent: #355 ## Problem `CalendarView` still owns view state, FullCalendar sync, optimistic recurring-instance state, drag/drop mutation, popover state, navigation, keybindings, and rendering in one large component. ## Solution Split behavior-preserving controller logic into focused hooks/components: - calendar navigation/view-mode state - keyboard handling - optimistic event/materialized recurrence state - keep `FcCalendar` and visible calendar behavior unchanged ## Acceptance criteria - `CalendarView` becomes a smaller composition layer. - Week/day/month navigation, drag/drop, recurrence edits, and popovers keep working. - Focused tests cover extracted pure/controller logic where practical.

barrettruth added the

track:ui

type:cleanup

labels

2026-05-11 21:02:07 +00:00

barrettruth referenced this issue

2026-05-11 21:02:45 +00:00

tracker: foundation hardening before Google sync #355

barrettruth referenced this issue

2026-05-12 13:17:40 +00:00

cleanup: make FullCalendar adapter the canonical calendar contract #390

barrettruth referenced this issue from a pull request that will close it,