-
Notifications
You must be signed in to change notification settings - Fork 0
Review 4320
#4320 · HelloOjasMutreja · aged-PR cleanup Step 3, batch 3, 2026-08-23
Verdict: approve — awaiting merge
Field.tsx declares and forwards labelID and isGroupLabel; Field.doc.mjs documented neither.
Documents both, adds a contract test and a patch changeset.
no API change (docs only)
no new theme targets
API no · Visual no · Theme no
Field.tsx:115 labelID?: string, :123 isGroupLabel?: boolean, :196 default false. The JSDoc at :117-122 says the label renders as a non-<label> <span> for a group — which is what the PR's description says.
approve. Docs-only, low risk, both props and their descriptions verified against source.
See the review on #4320. Attribution: [Reviewed by Robohands].
(posted directly by the loop under the aged-PR brief; nothing edited)