Skip to content
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.

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

[Feature]: add linting for forced synchronous layout #33803

Open
1 task done
spmonahan opened this issue Feb 7, 2025 · 1 comment
Open
1 task done

[Feature]: add linting for forced synchronous layout #33803

spmonahan opened this issue Feb 7, 2025 · 1 comment

Comments

@spmonahan
Copy link
Contributor

Area

React Components (@fluentui/react-components)

Describe the feature that you would like added

We should add a lint rule to prevent calling APIs that trigger forced synchronous layout.

Forced synchronous layout often causes performance issues in applications. Fluent should avoid triggering synchronous layout whenever possible.

Additional context

Performance degradation caused by forced synchronous layout has come up several times recently and #33665 is one example where forced synchronous layout has been removed.

We should add linting to prevent future instances of synchronous layout and identify existing instances so we can remove them.

Have you discussed this feature with our team

@Hotell @layershifter

Validations

  • Check that there isn't already an issue that requests the same feature to avoid creating a duplicate.

Priority

High

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

No branches or pull requests

4 participants