New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add guidance for layout-forcing apis #564

Open

xiaochengh wants to merge 1 commit into main from sync-layout-fixup

xiaochengh commented Mar 18, 2025 •

edited by pr-preview bot

Loading

Fixes #556

Explicitly calls out synchronous layout-forcing APIs as inappropriate
Suggests events or observers when an async API can't use Promises. This (implicitly) covers cases when layout is involved
Moves the "an API might need to be asynchronous" criteria to the previous section, since it's about when to use async API instead of how to design an async API


          Add guidance for layout-forcing apis

bc16b61

Signed-off-by: Xiaocheng Hu <[email protected]>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet