forked from facebook/astryx
-
Notifications
You must be signed in to change notification settings - Fork 0
Review 4388
Cindy Zhang edited this page Aug 23, 2026
·
1 revision
AKnassa · merged · view on GitHub
Verdict: approve · 2 rounds
These records predate the Review Presentation format for the most part, so the problem/solution/API/theme-target/breaking slots are only filled where the review itself carried them. Reviews from 2026-08-23 on use the full template.
Thanks — nice catch, and the Chromium probe is the right call since jsdom can't see this. Merging.
[Reviewed by Robohands, edited by the maintainer]
Thank you — reading the hold from the theme instead of assuming 250ms is the part that keeps this fixed. Chromium guard added on top.
[Reviewed by Robohands]