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
layout: Start work on table row height and vertical-align #31246
Conversation
🔨 Triggering try run (#7755378350) for Linux WPT layout-2020 |
Test results for linux-wpt-layout-2020 from try job (#7755378350): Flaky unexpected result (14)
Stable unexpected results that are known to be intermittent (16)
Stable unexpected results (19)
|
|
370f3ff
to
309a4c2
Compare
🔨 Triggering try run (#7757564533) for Linux WPT layout-2020 |
Test results for linux-wpt-layout-2020 from try job (#7757564533): Flaky unexpected result (13)
Stable unexpected results that are known to be intermittent (17)
Stable unexpected results (4)
|
|
309a4c2
to
aaa5c14
Compare
🔨 Triggering try run (#7813166812) for Linux WPT layout-2020 |
Test results for linux-wpt-layout-2020 from try job (#7813166812): Flaky unexpected result (22)
Stable unexpected results that are known to be intermittent (15)
|
✨ Try run (#7813166812) succeeded. |
layout | ||
.positioning_context | ||
.adjust_static_position_of_hoisted_fragments_with_offset( | ||
&content_rect.start_corner, | ||
&(&cell_content_rect.start_corner), |
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.
&(&cell_content_rect.start_corner), | |
&cell_content_rect.start_corner, |
components/layout_2020/flow/mod.rs
Outdated
match (self.inflow_baselines.first, box_fragment.baselines.first) { | ||
(None, Some(first)) => self.inflow_baselines.first = Some(first + box_block_offset), | ||
_ => {}, | ||
} |
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.
This was already the case, but BTW clippy would prefer
match (self.inflow_baselines.first, box_fragment.baselines.first) { | |
(None, Some(first)) => self.inflow_baselines.first = Some(first + box_block_offset), | |
_ => {}, | |
} | |
if let (None, Some(first)) = (self.inflow_baselines.first, box_fragment.baselines.first) { | |
self.inflow_baselines.first = Some(first + box_block_offset) | |
} |
This implements a very naive row height allocation approach. It has just enough to implement `vertical-align` in table cells. Rowspanned cells get enough space for their content, with the extra space necessary being allocated to the last row. There's still a lot missing here, including proper distribution of row height to rowspanned cells. Co-authored-by: Oriol Brufau <obrufau@igalia.com>
aaa5c14
to
9db9efc
Compare
This implements a very naive row height allocation approach. It has just enough to implement `vertical-align` in table cells. Rowspanned cells get enough space for their content, with the extra space necessary being allocated to the last row. There's still a lot missing here, including proper distribution of row height to rowspanned cells. Co-authored-by: Oriol Brufau <obrufau@igalia.com>
layout: Start work on table row height and vertical-align
This implements a very naive row height allocation approach. It has just
enough to implement
vertical-align
in table cells. Rowspanned cellsget enough space for their content, with the extra space necessary being
allocated to the last row. There's still a lot missing here, including
proper distribution of row height to rowspanned cells.
Co-authored-by: Oriol Brufau obrufau@igalia.com
./mach build -d
does not report any errors./mach test-tidy
does not report any errors