Skip to content

Wiki ingest: PR #568 Refactor stage 5 (slice 1): event-row verbs move to the environment#577

Open
avpv wants to merge 1 commit into
mainfrom
claude/confident-gates-ajz97e
Open

Wiki ingest: PR #568 Refactor stage 5 (slice 1): event-row verbs move to the environment#577
avpv wants to merge 1 commit into
mainfrom
claude/confident-gates-ajz97e

Conversation

@avpv

@avpv avpv commented Jun 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • Updated wiki/modules/views.md: corrected MenuBarView.swift line count 211→185 and added .environment(\.eventRowActions, eventRowActions) to body modifier chain description; corrected EventRowView.swift line count 721→647 and cluster counts (+HoverActions 169→198, +Title 102→104); added new EventRowActions entry to Event/task rows component list with full 17-verb inventory; updated +EventRow description to note it now houses both the eventRowActions value and eventRow() builder.
  • Updated wiki/concepts/menu-bar-popover.md: added EventRowActions environment pattern to the How-it's-wired section — one EventRowActions value built by MenuBarView+EventRow.swift:18 and installed at MenuBarView.body:134.
  • Appended one entry to wiki/log.md.

Trigger

Auto-generated by the wiki-ingest routine for merged PR #568 (9c79696).

Notes

Biggest corrections: (1) EventRowActions.swift added — new 17-verb environment carrier replaces individual closure parameters on EventRowView; documented in the Event/task rows component list. (2) EventRowView.swift line count 721→647; +HoverActions 169→198. (3) MenuBarView.swift line count 211→185 with .environment(\.eventRowActions, …) added to modifier chain. No pages flagged; §5 verification fully passed.


Generated by Claude Code

avpv commented Jun 11, 2026

Copy link
Copy Markdown
Owner Author

Auto-merge could not be enabled: the repository has no branch protection rules configured for main, which GitHub requires before squash auto-merge can be armed. All §5 verification checks passed and no pages were flagged — this PR is ready to merge manually.


Generated by Claude Code

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