Skip to content
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

Make `MAX_RECT` much larger. #290

Merged
merged 1 commit into from Jun 10, 2016
Merged

Conversation

@pcwalton
Copy link
Collaborator

pcwalton commented Jun 10, 2016

It was too small to cover many pages, which caused long pages to
disappear when scrolling down.

r? @glennw (or whoever)

It was too small to cover many pages, which caused long pages to
disappear when scrolling down.
@pcwalton
Copy link
Collaborator Author

pcwalton commented Jun 10, 2016

@bors-servo: r=glennw

@bors-servo
Copy link
Contributor

bors-servo commented Jun 10, 2016

📌 Commit 0ed3ecd has been approved by glennw

@bors-servo
Copy link
Contributor

bors-servo commented Jun 10, 2016

Testing commit 0ed3ecd with merge 1c68d65...

bors-servo added a commit that referenced this pull request Jun 10, 2016
Make `MAX_RECT` much larger.

It was too small to cover many pages, which caused long pages to
disappear when scrolling down.

r? @glennw (or whoever)
@bors-servo
Copy link
Contributor

bors-servo commented Jun 10, 2016

💔 Test failed - travis

@pcwalton
Copy link
Collaborator Author

pcwalton commented Jun 10, 2016

@bors-servo: r=glennw

@bors-servo
Copy link
Contributor

bors-servo commented Jun 10, 2016

💡 This pull request was already approved, no need to approve it again.

  • This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.
@bors-servo
Copy link
Contributor

bors-servo commented Jun 10, 2016

📌 Commit 0ed3ecd has been approved by glennw

@metajack
Copy link

metajack commented Jun 10, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Jun 10, 2016

💡 This pull request was already approved, no need to approve it again.

  • This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.
@bors-servo
Copy link
Contributor

bors-servo commented Jun 10, 2016

📌 Commit 0ed3ecd has been approved by metajack

@nox
Copy link
Member

nox commented Jun 10, 2016

@bors-servo retry

bors-servo added a commit that referenced this pull request Jun 10, 2016
Make `MAX_RECT` much larger.

It was too small to cover many pages, which caused long pages to
disappear when scrolling down.

r? @glennw (or whoever)
@bors-servo
Copy link
Contributor

bors-servo commented Jun 10, 2016

Testing commit 0ed3ecd with merge 6649ec0...

@bors-servo
Copy link
Contributor

bors-servo commented Jun 10, 2016

☀️ Test successful - travis

@bors-servo bors-servo merged commit 0ed3ecd into servo:master Jun 10, 2016
1 of 3 checks passed
1 of 3 checks passed
continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.