fix: replace typeof window checks with typeof document #516
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
currently the
@tanstack/react-virtual
package throwsuseLayoutEffect does nothing on the server
warnings during ssr when using a deno runtime. this is because deno has a global window object available for browser compatibility (though is being considered for removal denoland/deno#13367). also see here for further reading.this pr replaces all
typeof window
checks across all packages withtypeof document
instead which works in all environments.prior art: motiondivision/motion#1522