Phantomjs 2 #35
Closed
Phantomjs 2 #35
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
For some reason it wasn't loading for me and thus the callback was never called. We don't need jQuery to do those calculations anyway so clean it up by removing it.
Happens when div loading is dependent on resources loading.
- Instead of modifying the height of the body (and html) after render we just set the mask to take care of it. The image we were getting was of the full size page cropped down to the height and the rest filled with white. I think this is because the larger initial page size caused the viewport to expand but then it didn’t contract. - Since args.div will supersede the mask eventually, don’t set the mask up front if args.div is present. Set the mask solely on the height arg as we can default top and left to 0. This fixes the failing unit tests when running locally.
|
Never mind, I worked out what was wrong. It was an issue with the version of bundler on TravisCI. @maxwell It's over to you now. |
I think we are striking this issue: rubygems/bundler#3558
steakunderscore
force-pushed
the
phantomjs-2
branch
from
c0bb405 to
d5b8a0e
Compare
| @@ -3,4 +3,6 @@ rvm: | |||
| - 2.0 | |||
| - 2.1 | |||
| - 2.2 | |||
| before_install: | |||
| - gem update bundler | |||
There was a problem hiding this comment.
Just a bit of googling for kittens ;)
On 22 Apr 2016 5:17 p.m., "Jeremy Kallman" [email protected] wrote:
In .travis.yml
#35 (comment):@@ -3,4 +3,6 @@ rvm:
- 2.0
- 2.1
- 2.2
+before_install:
- gem update bundler
Nice! How did you figure that out?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/maxwell/screencap/pull/35/files/0478476c539d753644599f4f679ba0ab546f2f99..d5b8a0e17ecd40a5fa38faaa71e08d916d591995#r60804388
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is not needed, and is only causing issues. The dependency on
>= 1.8.1.1is for thePhantomjs#runmethod.