Reviews

All reviews

Every PR this loop has reviewed, newest first. One page each: what the PR did, what the review said, and how it was judged.

New reviews are recorded here as they are drafted — R17. The page skeleton is on Review Presentation: problem, solution, API, theme targets, ossification, breaking, judgement, the review as posted, and what changed before posting.

39 PRs · 51 reviews posted

PR	author	verdict	rounds	state
#5354 docs: replace em dashes and curly quotes in CLI	josephfarina	approve	1	open
#5331 docs(cli): document the integration manifest doc	josephfarina	approve	1	open
#5330 docs(Avatar): restore the missing dense best-pra	josephfarina	approve	1	merged
#5329 docs: replace em dashes in MediaTheme and cli-in	josephfarina	approve	1	merged
#5317 fix(Table): keep group headers out of the caller	ernestt	approve	1	open
#5306 fix(Calendar): open on a month inside the min/ma	imdreamrunner	approve	1	open
#5292 docs(ComplexSelector): backfill two missing de	josephfarina	approve	1	merged
#5291 docs: replace em dashes in useScrollLock, Tool	josephfarina	approve	1	merged
#5251 docs: replace em dashes in theme and Selector	josephfarina	approve	1	merged
#5210 docs: replace em dashes in DateRangeInput and	josephfarina	approve	1	merged
#5201 feat(core): promote Stepper from Lab	ernestt	approve	3	open
#5200 feat(lab): harden ListInput to the states its	ernestt	comment	1	merged
#5197 feat(internal): lab readiness audit tooling +	ernestt	approve	1	merged
#5195 fix(i18n): use provider locale outside date in	nynexman4464	approve	2	merged
#5194 feat(i18n): expose provider-backed locale hook	nynexman4464	approve	1	merged
#5191 fix: Escape dismisses only the top-most overla	HelloOjasMutreja	comment	1	open
#5183 feat(core): let CheckboxInput and Switch name	freddymeta	request-changes	1	open
#5181 fix(NumberInput): track themed padding so the	freddymeta	approve	2	merged
#5180 docs: replace em dashes in theme-build, TextAr	josephfarina	approve	1	merged
#5179 feat(Selector): add selector-option-row theme	freddymeta	approve	1	merged
#5174 fix(core): close only the top-most layer on Es	AKnassa	comment	1	closed
#5171 feat(lint): require provider-backed locale API	nynexman4464	approve	1	merged
#5148 feat(DateTimeInput): expose toggle-icon and cl	freddymeta	approve	1	merged
#5147 fix(core): keep tooltip status button interact	freddymeta	approve	1	merged
#5145 feat(DateRangeInput): add maxRangeSpan/minRang	freddymeta	approve	2	merged
#5144 audit(Blockquote): drop the contentinfo landma	cixzhang	comment	1	merged
#5143 fix(RadioListItem): make the whole row a click	freddymeta	approve	2	merged
#5142 docs(banner): backfill two missing dense best-	josephfarina	approve	1	merged
#5110 fix(NumberInput): reject non-numeric character	freddymeta	request-changes	1	open
#5035 fix(SegmentedControl): clamp item labels to a	ernestt	comment	2	merged
#5018 fix(core): stop popup triggers from fighting t	cixzhang	comment	2	open
#4930 feat(core): conditional theme layer in defineT	cixzhang	comment	1	open
#4881 feat(core): one shared dismissal stack for eve	cixzhang	comment	1	open
#4865 fix(dialog): close only the top-most modal on	freddymeta	comment	1	closed
#4791 feat(FormLayout): add defaultOptionality — mar	freddymeta	approve	2	merged
#4500 feat(core): DialogHeroHeader — hero-style Dial	Lee-Dongwook	comment	1	closed
#4388 fix(MobileNav): closing the drawer could leave	AKnassa	approve	2	merged
#4327 feat(Avatar): add shape prop (circle	rounded	HelloOjasMutreja	request-changes	2
#4101 chore(deps): prune stale pnpm overrides, move	Han5991	approve	2	merged