Can't seem to get scrollToElement to work about capture-website HOT 6 CLOSED

The element must be loaded when the page is loaded. If it's loaded dynamically/lazily, you must use the waitForElement option.

I cannot really help more than that without code that reproduces the issue.

from capture-website.

I don't believe that the elements are loaded lazily but I am using waitForElement here's the code:

(it appeared strangely when I copied and pasted it so here's an image instead)

from capture-website.

Try using

E.G.:

 // code here 

syntax in order to let us test your code. Or add it to a pen/jsfiddle.

from capture-website.

I have the same problem.

const captureWebsite = require('capture-website');

(async () => {
  await captureWebsite.file('https://github.com/sindresorhus/capture-website', 'screenshot.png', { scrollToElement: "#readme" });
})()

This takes a screenshot from the top of the page.

Node version 12.18.3, capture-website 1.2.1.
What am I doing wrong?

(Edit: attached screenshot)

from capture-website.

I believe this is the issue: https://github.com/sindresorhus/capture-website/blob/master/index.js#L71-L75

Before we try to scroll the parentElement we should also scroll the body so that the parent element is visible. Currently we try to scroll elements that are not scrollable and/or outside of the viewport.

parentElement.scrollIntoView(true) something like this is missing

from capture-website.

Thanks for the research. It worked for me so I created a PR.

from capture-website.