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.

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

[Dialog] Improve performance #2855

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[Dialog] Improve performance #2855

wants to merge 2 commits into from
+7 −2

Conversation

andresz1
Copy link

@andresz1 andresz1 commented Apr 19, 2024 •
edited

Description

This PR improves Dialog component INP performance by prioritising paint when modal prop is set to true

  • The style change over the body is now executed using a setTimeout to avoid doing synchronously before the paint.

Before

Screenshot 2024-04-19 at 12 03 15

After

Screenshot 2024-04-19 at 11 59 43

@andresz1 andresz1 changed the title Improve dialog performance [Dialog] Improve performance Apr 19, 2024
nucliweb
nucliweb approved these changes Apr 23, 2024
View reviewed changes
Copy link

@nucliweb nucliweb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

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

@nucliweb nucliweb nucliweb approved these changes

@andresin87 andresin87 andresin87 approved these changes

@kikoruiz kikoruiz kikoruiz approved these changes

@benoitgrelard benoitgrelard Awaiting requested review from benoitgrelard benoitgrelard is a code owner

@andy-hook andy-hook Awaiting requested review from andy-hook andy-hook is a code owner

@vladmoroz vladmoroz Awaiting requested review from vladmoroz vladmoroz is a code owner

@lucasmotta lucasmotta Awaiting requested review from lucasmotta lucasmotta is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@andresz1 @nucliweb @andresin87 @kikoruiz