Feature: ability to use panzoom with SSR #499
Comments
|
Thanks for opening an issue. It makes sense to be able to load it without having to dynamically load it so I'll add the window checks, but you should still only initialize Panzoom on the front-end, such as in a |
|
By the way, this is how you would include something like panzoom in next.js: https://nextjs.org/docs/advanced-features/dynamic-import#with-no-ssr But this shouldn't be necessary once I finish the patch. |
|
Thanks for the answer! I see that you've just applied the fix for this issue. Have you deployed it to npm? |
timmywil
added a commit
that referenced
this issue
Jul 21, 2020
## [4.3.1](4.3.0...4.3.1) (2020-07-21) ### Bug Fixes * check for window and document to allow ssr loading ([3a0119e](3a0119e)), closes [#499](#499)
|
🎉 This issue has been resolved in version 4.3.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
timmywil
added a commit
that referenced
this issue
Aug 6, 2020
timmywil
added a commit
that referenced
this issue
Aug 6, 2020
## [4.3.1](4.3.0...4.3.1) (2020-07-21) ### Bug Fixes * check for window and document to allow ssr loading ([3a0119e](3a0119e)), closes [#499](#499)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What problem does this feature solve?
This feature solves crashes of next.js (or any other SSR app) on dev & build modes
Describe the solution you'd like
Try to check typeof of
windowor use exenv's canUseDom utilityAdditional context
Here comes the log output of
npm run devinside next.js applog.txt
The text was updated successfully, but these errors were encountered: