mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-10-23 02:05:42 +00:00
Backed out 2 changesets (bug 1591925) for reftest failures on flexbox-min-bsize-keywords-vert-1.html
Backed out changeset eb9d579fe07c (bug 1591925) Backed out changeset 7d0f795c10a4 (bug 1591925)
This commit is contained in:
parent
683ac9e167
commit
8719db1dde
@ -2093,18 +2093,18 @@ void FlexItem::CheckForMinSizeAuto(const ReflowInput& aFlexItemReflowInput,
|
||||
// the flex container's main axis) iff:
|
||||
// (a) its computed value is "auto"
|
||||
// (b) the "overflow" sub-property in the same axis (the main axis) has a
|
||||
// computed value of "visible" or does not create a scroll container.
|
||||
// computed value of "visible"
|
||||
const auto& mainMinSize = aAxisTracker.IsRowOriented()
|
||||
? pos->MinISize(aAxisTracker.GetWritingMode())
|
||||
: pos->MinBSize(aAxisTracker.GetWritingMode());
|
||||
|
||||
// If the scrollable overflow makes us create a scroll container, then we
|
||||
// don't need to do any extra resolution for our `min-size:auto` value.
|
||||
// We don't need to check for scrollable overflow in a particular axis
|
||||
// because this will be true for both or neither axis.
|
||||
// NOTE: Technically we should be checking the 'overflow' subproperty in the
|
||||
// main axis. But since we only care whether it's 'visible', we can check
|
||||
// either subproperty -- because they must be BOTH 'visible' or BOTH
|
||||
// non-'visible' due to the way the subproperties interact.
|
||||
mNeedsMinSizeAutoResolution =
|
||||
IsAutoOrEnumOnBSize(mainMinSize, IsInlineAxisMainAxis()) &&
|
||||
disp->IsScrollableOverflow();
|
||||
disp->mOverflowX == StyleOverflow::Visible;
|
||||
}
|
||||
|
||||
nscoord FlexItem::GetBaselineOffsetFromOuterCrossEdge(
|
||||
|
@ -1,14 +0,0 @@
|
||||
<!DOCTYPE html>
|
||||
<title>Reference: Select element in flexbox with zero height</title>
|
||||
<link rel="author" title="Mozilla" href="https://mozilla.org">
|
||||
<style>
|
||||
.container {
|
||||
height: 0px;
|
||||
border: 1px dotted black;
|
||||
}
|
||||
</style>
|
||||
<body>
|
||||
<div class="container">
|
||||
<select style="width:100%"><option>Hi</option></select>
|
||||
</div>
|
||||
</body>
|
@ -1,19 +0,0 @@
|
||||
<!DOCTYPE html>
|
||||
<title>Select element in flexbox with zero height</title>
|
||||
<link rel="author" title="Mozilla" href="https://mozilla.org">
|
||||
<link rel="help" href="https://bugzilla.mozilla.org/show_bug.cgi?id=1591925">
|
||||
<link rel="help" href="https://drafts.csswg.org/css-flexbox/#content-size-suggestion">
|
||||
<link rel="match" href="select-element-zero-height-001-ref.html">
|
||||
<style>
|
||||
.container {
|
||||
display: flex;
|
||||
flex-direction: column;
|
||||
height: 0px;
|
||||
border: 1px dotted black;
|
||||
}
|
||||
</style>
|
||||
<body>
|
||||
<div class="container">
|
||||
<select><option>Hi</option></select>
|
||||
</div>
|
||||
</body>
|
@ -1,19 +0,0 @@
|
||||
<!DOCTYPE html>
|
||||
<title>Reference: Select element with background color in flexbox with zero height</title>
|
||||
<link rel="author" title="Mozilla" href="https://mozilla.org">
|
||||
<style>
|
||||
.container {
|
||||
height: 0px;
|
||||
margin-bottom: 50px;
|
||||
border: 1px dotted black;
|
||||
}
|
||||
.with-background {
|
||||
width: 100%;
|
||||
background: lime;
|
||||
}
|
||||
</style>
|
||||
<body>
|
||||
<div class="container">
|
||||
<select class="with-background"><option>Hi</option></select>
|
||||
</div>
|
||||
</body>
|
@ -1,22 +0,0 @@
|
||||
<!DOCTYPE html>
|
||||
<title>Select element with background color in flexbox with zero height</title>
|
||||
<link rel="author" title="Mozilla" href="https://mozilla.org">
|
||||
<link rel="help" href="https://bugzilla.mozilla.org/show_bug.cgi?id=1591925">
|
||||
<link rel="help" href="https://drafts.csswg.org/css-flexbox/#content-size-suggestion">
|
||||
<link rel="match" href="select-element-zero-height-002-ref.html">
|
||||
<style>
|
||||
.container {
|
||||
display: flex;
|
||||
flex-direction: column;
|
||||
height: 0px;
|
||||
border: 1px dotted black;
|
||||
}
|
||||
.with-background {
|
||||
background: lime;
|
||||
}
|
||||
</style>
|
||||
<body>
|
||||
<div class="container">
|
||||
<select class="with-background"><option>Hi</option></select>
|
||||
</div>
|
||||
</body>
|
Loading…
Reference in New Issue
Block a user