-
-
Notifications
You must be signed in to change notification settings - Fork 358
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
Take full page screenshots #143 #148
Take full page screenshots #143 #148
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
7d6960e
to
4cc042e
Compare
Added the fullPage flag to take full screen screenshots updated the UI accordingly to properly show the screenshots instead of scaling it down
4cc042e
to
d7833f9
Compare
First of all, again thanks a lot for taking the time and implementing that feature you requested! :) I tried it locally, but it seems that for some reason, sometimes, it results into the crawler getting stuck fetching that page. for example when fetching that link, it always times out which doesn't happen when full page screenshot is enabled:
I don't know how puppeteer implements this, but I wonder if it misbehaves if the page expands on scrolling for example or something. |
Thanks, I'll check it out. |
Are you running that with the docker container for chrome or just with the worker? I tried that out many times now and never had any issues. |
Ok, no worries. Let's merge it and I'll add some protections so that taking screenshots timing out doesn't fail the entire crawling :) |
Added the fullPage flag to take full screen screenshots updated the UI accordingly to properly show the screenshots instead of scaling it down