qentinelqi / qweb

Keyword driven automation for the web
https://github.com/qentinelqi/qweb
Apache License 2.0
40 stars 17 forks source link

Full page screenshot #72

Closed tkoukkari closed 2 years ago

tkoukkari commented 2 years ago

Implements issue #61

We decided to enable this only for browsers that support full page screenshots natively (Firefox at the moment). We could take multiple screenshots and scroll in between + stitch the images together at the end, but this will not work too well if page has dynamic elements that move with scrolling.

kalyncoose commented 2 years ago

Totally understandable, will work nicely in QWeb once all major browsers support that natively. Thank you for looking into it and implementing what you could!

tkoukkari commented 2 years ago

Totally understandable, will work nicely in QWeb once all major browsers support that natively. Thank you for looking into it and implementing what you could!

I will add Chrome & Edge support too. Found another way which seems to be good enough.

tkoukkari commented 2 years ago

Example screenshots with Chrome:

Normal: screenshot-RFDEBUG_REPL-e01e221e-8e9c-4af9-b338-8313ff9669b2

Full page: screenshot-RFDEBUG_REPL-8d2562a5-cb2d-4cb6-8fb3-cdc55e566964