Skip to content

Use the correct container size in compute_overflow#7771

Merged
bors-servo merged 1 commit intoservo:masterfrom
mbrubeck:rtl-stacking-context
Sep 29, 2015
Merged

Use the correct container size in compute_overflow#7771
bors-servo merged 1 commit intoservo:masterfrom
mbrubeck:rtl-stacking-context

Conversation

@mbrubeck
Copy link
Copy Markdown
Contributor

Fixes #7768 - positioning of RTL stacking contexts. r? @pcwalton

Review on Reviewable

Fixes servo#7768 - positioning of RTL stacking contexts.
@highfive highfive added the S-awaiting-review There is new code that needs to be reviewed. label Sep 28, 2015
@pcwalton
Copy link
Copy Markdown
Contributor

@bors-servo: r+

@bors-servo
Copy link
Copy Markdown
Contributor

📌 Commit 441cc47 has been approved by pcwalton

@highfive highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-awaiting-review There is new code that needs to be reviewed. labels Sep 29, 2015
@bors-servo
Copy link
Copy Markdown
Contributor

⌛ Testing commit 441cc47 with merge 8009030...

bors-servo pushed a commit that referenced this pull request Sep 29, 2015
Use the correct container size in compute_overflow

Fixes #7768 - positioning of RTL stacking contexts. r? @pcwalton

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7771)
<!-- Reviewable:end -->
@bors-servo
Copy link
Copy Markdown
Contributor

💔 Test failed - mac-rel-wpt

@highfive highfive added S-tests-failed The changes caused existing tests to fail. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Sep 29, 2015
@mbrubeck
Copy link
Copy Markdown
Contributor Author

@bors-servo retry

Looks like a GitHub/infra issue

@bors-servo
Copy link
Copy Markdown
Contributor

⌛ Testing commit 441cc47 with merge e68bd8d...

bors-servo pushed a commit that referenced this pull request Sep 29, 2015
Use the correct container size in compute_overflow

Fixes #7768 - positioning of RTL stacking contexts. r? @pcwalton

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7771)
<!-- Reviewable:end -->
@highfive highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-tests-failed The changes caused existing tests to fail. labels Sep 29, 2015
@bors-servo
Copy link
Copy Markdown
Contributor

☀️ Test successful - android, gonk, linux-dev, linux-rel, mac-dev-ref-unit, mac-rel-css, mac-rel-wpt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants