-
Notifications
You must be signed in to change notification settings - Fork 0
Review 5488
josephfarina · internal (.github/ENGOWNERS:20) · #5488 · MERGED 2026-08-25
Head 45adafb4f6bdaefb797b056bcf892abc1f371253. Reviewed as part of a three-PR docs batch with #5490 and #5413.
1 decision · 0 runtime lines of 5 — five em dashes in doc description strings become colon / semicolon / comma. All slots clear.
The one hazard in a mechanical text sweep is hitting a string it should not. Checked: every changed line is a description value — no name, no displayName, no code fence, no example string, no locale catalog entry. DateInput.doc.mjs:437 correctly keeps its CJK …….
Not blocking: the dense nativePicker line loses the contrast the dash carried — "…shown by the OS picker, refused on commit instead" now reads as one more list item rather than the exception (DateInput.doc.mjs:555). Inline left.
Risk class: low-risk.