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

Floats should be rendered in front of in-flow blocks #677

Closed
sanxiyn opened this issue Aug 6, 2013 · 3 comments
Closed

Floats should be rendered in front of in-flow blocks #677

sanxiyn opened this issue Aug 6, 2013 · 3 comments
Milestone

Comments

@sanxiyn
Copy link
Contributor

@sanxiyn sanxiyn commented Aug 6, 2013

CSS 2.1 section 9.5 language is: "A float can overlap other boxes in the normal flow... When this happens, floats are rendered in front of non-positioned in-flow blocks, but behind in-flow inlines."

In terms of Appendix E, Servo currently does 4 and 5 together. This is visible in test_float_placement.html, for example. Placement is correct, but blue background float should be rendered in front of gray background block.

@larsbergstrom
Copy link
Contributor

@larsbergstrom larsbergstrom commented Dec 10, 2013

Needed for #1366.

@jdm
Copy link
Member

@jdm jdm commented Apr 18, 2014

If it was needed for acid2, does that imply that this is fixed?

ChrisParis pushed a commit to ChrisParis/servo that referenced this issue Sep 7, 2014
Rename embedded-content-0 and -1 to their new (pretty) ids
@pcwalton
Copy link
Contributor

@pcwalton pcwalton commented Sep 11, 2014

Fixed.

@pcwalton pcwalton closed this Sep 11, 2014
glennw pushed a commit to glennw/servo that referenced this issue Jan 16, 2017
Drop unneeded Arc

The Arc seems unnecessary here.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/677)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

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