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

perf(cdk/table): Further defer direct dom measurement. In all cases I… #29936

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kseamon
Copy link
Collaborator

@kseamon kseamon commented Oct 28, 2024

…'ve observed, this fully eliminates layout thrashing from table init.

@kseamon kseamon added the G This is is related to a Google internal issue label Oct 28, 2024
@kseamon kseamon force-pushed the table-more-resize-observer branch 3 times, most recently from 677a0b2 to 4192088 Compare October 31, 2024 14:54
…'ve observed, this fully eliminates layout thrashing from table init.
@kseamon kseamon force-pushed the table-more-resize-observer branch from 548f9cc to d044043 Compare October 31, 2024 16:54
@kseamon kseamon requested a review from andrewseguin October 31, 2024 17:13
@kseamon kseamon marked this pull request as ready for review October 31, 2024 17:13
@kseamon kseamon requested a review from a team as a code owner October 31, 2024 17:13
@kseamon kseamon requested review from amysorto and removed request for a team October 31, 2024 17:13
@andrewseguin andrewseguin self-assigned this Dec 4, 2024
@andrewseguin andrewseguin added the action: merge The PR is ready for merge by the caretaker label Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker G This is is related to a Google internal issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants