Skip to content
This repository has been archived by the owner on Mar 22, 2018. It is now read-only.

waitFor and waitTimeout : issue #39 #40

Closed
wants to merge 1 commit into from
Closed

waitFor and waitTimeout : issue #39 #40

wants to merge 1 commit into from

Conversation

tcoulter
Copy link

Add the waitFor and waitTimeout parameters. waitFor will be evaluated periodically within the phantomjs browser; if specified, screenshot-as-a-service won't take the screenshot until waitFor evaluates to true. Example: "function() {return Config.get('loaded') == true}" -- note, you must URI encode it. waitForTimeout specifies a maximum time to wait before giving up and rendering the screenshot anyway.

I'm sure there's more I have to do, like update the readme. If you can take a look to see if it's functionally sound beforehand I'd appreciate it.

…will be evaluated periodically within the phantomjs browser; if specified, screenshot-as-a-service won't take the screenshot until waitFor evaluates to true. Example: "function() {return Config.get('loaded') == true}" -- note, you must URI encode it. waitForTimeout specifies a maximum time to wait before giving up and rendering the screenshot anyway.
@tcoulter tcoulter closed this May 26, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant