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
feat: promisify win.capturePage() #15743
Conversation
This should go through our deprecation cycle right? We should probably add a JS side wrapper so that callback-style still works but with a deprecation warning |
1020f35
to
5c5286b
Compare
d2bf372
to
7063cd9
Compare
@MarshallOfSound rebased and updated! |
[NativeImage](native-image.md) that stores data of the snapshot. Omitting | ||
`rect` will capture the whole visible page. | ||
be called with `callback(image)`. The `image` is an instance of [NativeImage](native-image.md) | ||
that stores data of the snapshot. Omitting `rect` will capture the whole visible page. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that stores data of the snapshot
could be clearer, but that's the only thing I could find with this PR. The rest looks ✨ and it's honestly a nit.
d2e7de2
to
848defd
Compare
Release Notes Persisted
|
Description of Change
This PR promisifies
win.capturePage
.Todo:
/cc @ckerr @deepak1556
Checklist
npm test
passesRelease Notes
Notes: promisify win.capturePage()
Co-authored-by: Charles Kerr ckerr@github.com