layout: Properly propagate baselines of flex items that do not participate in baseline alignment#42038
Conversation
| This does introduce a few new test failures but from what I can work out these are actual bugs that were previously hidden by the existing behaviour? |
| 🔨 Triggering try run (#21200578750) for Linux (WPT) |
| Test results for linux-wpt from try job (#21200578750): Flaky unexpected result (28)
Stable unexpected results that are known to be intermittent (33)
Stable unexpected results (7)
|
| |
4cca2c8 to 81096db Compare 18f5172 to b34e158 Compare | Note to self, push up the code accounting for flex direction reversal and simplifications after discussion with Martin. |
6b991a7 to e875ea6 Compare | fragments: Vec<Fragment>, | ||
| positioning_context: PositioningContext, | ||
| | ||
| content_baselines_relative_to_margin_box: Baselines, |
There was a problem hiding this comment.
Can you add some rustdoc explaining the purpose of this field and how it differs from the other baseline?
There was a problem hiding this comment.
Hopefully something like I've added is right?
e875ea6 to 9109333 Compare 9109333 to e7c5cc8 Compare | There are some new failures here, but I suspect they are just exposing some of the existing bugs we discussed in our call. @lukewarlow can you please just update the expected results and then we can address the new failures later? |
Signed-off-by: Luke Warlow <lwarlow@igalia.com>
Signed-off-by: Luke Warlow <lwarlow@igalia.com> Co-authored-by: Martin Robinson <mrobinson@igalia.com>
Signed-off-by: Luke Warlow <lwarlow@igalia.com>
Signed-off-by: Luke Warlow <lwarlow@igalia.com>
e7c5cc8 to 2dd4757 Compare | 🔨 Triggering try run (#21446495074) for Linux (WPT) |
| Test results for linux-wpt from try job (#21446495074): Flaky unexpected result (33)
Stable unexpected results that are known to be intermittent (32)
|
| ✨ Try run (#21446495074) succeeded. |
Fix baseline propagation from flex containers
Flex item layout previously conflated baselines that got propagated and baseline alignment within flex.
Testing: Before and after layout of the manual test cases.
test.html (from original issue)
test2.html (from Oriol's comment)
Fixes: #41905