-
Notifications
You must be signed in to change notification settings - Fork 271
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
Page layout metrics #43
Conversation
Hi @kaero598, I'll just add one more scenario in unit tests because you're testing the window size, but you don't test the case where the layout is larger than the window. |
@kaero598 I added a comment, I'll wait for your response before I go ahead |
@kaero598 Thanks, but I did it already, but didn't push yet as I was waiting for your answer about the LayoutMetricFailed excption |
@gsouf That's was confusing because I didn't see your comment. Where can I find it? |
Can you see the review here https://github.com/chrome-php/headless-chromium-php/pull/43/files#diff-472fd206a8e051d0df4d751280094c81R35 |
Nope. I see code changes and no comments. Your link just highlights the line before exception is thrown. |
Sorry I never used that new review tool. I guess you can see it now |
The missing
Page.getLayoutMetrics
feature is the only thing that prevents taking full-size screenshots.