Skip to content

Conversation

sebmarkbage
Copy link
Collaborator

The index is both used as the key and for hydration purposes. Previously we didn't preserve the index when sorting so the index didn't line up which caused hydration to be the wrong slot when sorted.

@sebmarkbage sebmarkbage requested a review from eps1lon October 13, 2025 03:06
@meta-cla meta-cla bot added the CLA Signed label Oct 13, 2025
@github-actions github-actions bot added the React Core Team Opened by a member of the React Core Team label Oct 13, 2025
@sebmarkbage sebmarkbage merged commit d7215b4 into facebook:main Oct 13, 2025
246 of 247 checks passed
sebmarkbage added a commit that referenced this pull request Oct 13, 2025
Stacked on #34829.

This lets you get an overview more easily when there's lots of things
like scripts downloading. Pluralized the name. E.g. `script` ->
`scripts` or `fetch` -> `fetches`.

This only groups them consecutively when they'd have the same place in
the list anyway because otherwise it might cover up some kind of
waterfall effects.

<img width="404" height="225" alt="Screenshot 2025-10-13 at 12 06 51 AM"
src="https://github.com/user-attachments/assets/da204a8e-d5f7-4eb0-8c51-4cc5bfd184c4"
/>

Expanded:

<img width="407" height="360" alt="Screenshot 2025-10-13 at 12 07 00 AM"
src="https://github.com/user-attachments/assets/de3c3de9-f314-4c87-b606-31bc49eb4aba"
/>
sebmarkbage pushed a commit to vercel/next.js that referenced this pull request Oct 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants