Skip to content
This repository has been archived by the owner on Jul 12, 2020. It is now read-only.

Add support for delays #29

Closed
wants to merge 3 commits into from
Closed

Add support for delays #29

wants to merge 3 commits into from

Conversation

addyosmani
Copy link
Owner

Changes:

  • API now supports a delay (in seconds) with a test
  • Code formatting

reviewer: @sindresorhus

### Set a custom delay before capturing the page

Type: Number (seconds)
Default: 0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe default to 1 second to fix most problems with it not being able to capture JS changes right after document load.

@addyosmani
Copy link
Owner Author

@sindresorhus PR updated per last comment :)

@@ -23,6 +23,7 @@ var cli = meow({
'Options',
' --verbose Displays more information',
' --viewport-size=1024x768 Sets the viewport size'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing comma

@addyosmani addyosmani closed this in 82c9dfd Jan 4, 2015
addyosmani added a commit that referenced this pull request Jan 4, 2015
@sindresorhus sindresorhus deleted the delay branch January 4, 2015 17:30
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

2 participants