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
Import victor's layout system 🍷 #24165
Conversation
Heads up! This PR modifies the following files:
|
d159564
to
a7526ba
Compare
@@ -352,12 +393,14 @@ impl Display { | |||
match self.outside() { | |||
DisplayOutside::Inline => { | |||
let inside = match self.inside() { | |||
#[cfg(any(feature = "servo-layout-2013", feature = "gecko"))] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#[cfg]
ing this out is wrong, why?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't keep it, I removed DisplayInside::Block
and DisplayInside::Inline
. Those are hacks to make layout supposedly easier, but it's just hindering layout 2020.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added back a blockification which makes it store DisplayInside::FlowRoot
with servo-layout-2020
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Err, I meant I made it store DisplayInline::Flow
, sorry.
r+ with nits Thanks @nox! |
It doesn't construct flows anymore.
@bors-servo r=SimonSapin |
📌 Commit 4444c5a has been approved by |
Import victor's layout system 🍷 <!-- Reviewable:start --> This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/24165) <!-- Reviewable:end -->
☀️ Test successful - linux-rel-css, linux-rel-wpt, status-taskcluster |
servo/servo#24165 Differential Revision: https://phabricator.services.mozilla.com/D45740 --HG-- extra : moz-landing-system : lando
servo/servo#24165 Differential Revision: https://phabricator.services.mozilla.com/D45740 UltraBlame original commit: ff80bc57d655e8da75c74e99830ff3346c0bbb1c
servo/servo#24165 Differential Revision: https://phabricator.services.mozilla.com/D45740 UltraBlame original commit: ff80bc57d655e8da75c74e99830ff3346c0bbb1c
servo/servo#24165 Differential Revision: https://phabricator.services.mozilla.com/D45740 UltraBlame original commit: ff80bc57d655e8da75c74e99830ff3346c0bbb1c
This change is