Skip to content

Cap height of slow list demo previews#8517

Open
TunaDev0 wants to merge 1 commit into
reactjs:mainfrom
TunaDev0:fix-8486-cap-example-height
Open

Cap height of slow list demo previews#8517
TunaDev0 wants to merge 1 commit into
reactjs:mainfrom
TunaDev0:fix-8486-cap-example-height

Conversation

@TunaDev0

@TunaDev0 TunaDev0 commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Summary

Closes issue #8486

  • add a max height and scrolling to the .items container in the artificially slow useDeferredValue list demos
  • add a max height and scrolling to the .items container in the artificially slow useTransition posts demos
  • keep the existing item counts and slowdown behavior unchanged, while preventing the preview output from pushing the rest of the page too far down

Testing

  • pre-commit hooks passed during commit creation

Why this better:

  1. Names actual change
  2. Mentions both pages
  3. States no behavior change
  4. Matches issue intent closely
    If you want closer to issue wording, use:

Involved routes:

  • /reference/react/useDeferredValue
  • /reference/react/useTransition

Example of improvement

Left: PR changes
Right: current implementation
The current implementation can expand the list to around 12500px in height.
image

@github-actions

github-actions Bot commented Jul 5, 2026

Copy link
Copy Markdown

Size changes

Details

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant