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.

Sign up for GitHub

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

Jump to bottom

Absolutely-positioned elements break inline flows into two #10642

Closed
pcwalton opened this issue Apr 15, 2016 · 0 comments
Closed

Absolutely-positioned elements break inline flows into two #10642

pcwalton opened this issue Apr 15, 2016 · 0 comments
Assignees
@pcwalton
Labels
A-layout/block A-layout/inline A-layout/tree-builder C-assigned There is someone working on resolving the issue C-has-manual-testcase I-wrong An incorrect behaviour is observed.

Comments

@pcwalton
Copy link
Contributor

Test case:

<!DOCTYPE html>
x<div style="position: absolute;"></div>x

This should display "xx" but has two paragraphs in Servo.

Breaks YouTube.
@notriddle notriddle self-assigned this Apr 15, 2016
@notriddle notriddle added the C-assigned There is someone working on resolving the issue label Apr 15, 2016
@pcwalton pcwalton assigned pcwalton and unassigned notriddle Apr 15, 2016
pcwalton added a commit to pcwalton/servo that referenced this issue Apr 16, 2016
@pcwalton
layout: Make absolutely-positioned blocks not generate {ib} splits.
e840c7a 
Improves YouTube.

Closes servo#10642.
@pcwalton pcwalton mentioned this issue Apr 16, 2016
Merged
bors-servo pushed a commit that referenced this issue Apr 16, 2016
Auto merge of #10644 - pcwalton:absolute-ib-split, r=mbrubeck
86778a0 
layout: Make absolutely-positioned blocks not generate {ib} splits.

Improves YouTube.

Closes #10642.

r? @mbrubeck

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

@pcwalton pcwalton

Labels
A-layout/block A-layout/inline A-layout/tree-builder C-assigned There is someone working on resolving the issue C-has-manual-testcase I-wrong An incorrect behaviour is observed.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pcwalton @notriddle