118639 2013-07-12 23:56:50 -0700 A (column) flexbox should take min-height into account for sizing 2022-08-26 01:28:58 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Unspecified Unspecified RESOLVED CONFIGURATION CHANGED BlinkMergeCandidate P2 Normal --- 1 rniwa webkit-unassigned ahmad.saleem792 ap bdakin bfulgham dino hyatt simon.fraser zalan oldest_to_newest 908139 0 rniwa 2013-07-12 23:56:50 -0700 Consider merging https://chromium.googlesource.com/chromium/blink/+/1a8ed26a737c8eb3544e5241d638c872f2a0058a Per http://dev.w3.org/csswg/css-flexbox/#intrinsic-sizes "The max-content main size of a flex container is the sum of the flex container's items' max-content contributions. The min-content main size of a single-line flex container is the sum of the flex container's items' min-content contributions" "The main-size min-content/max-content contribution of a flex item is its outer hypothetical main size when sized under a min-content/max-content constraint (respectively)." plus 9.2: "The hypothetical main size is the item's flex base size clamped according to its min and max main size properties." Therefore, when we calculate containerMainInnerSize, we should use sumHypotheticalMainSize instead of sumFlexBaseSize. 1893939 1 ahmad.saleem792 2022-08-26 01:28:45 -0700 This patch is not required because Webkit GitHub Code already matches with this one: https://github.com/WebKit/WebKit/blob/4ccdb2ac7f7c989dd9d92ec1c42dccc44ef21b6a/Source/WebCore/rendering/RenderFlexibleBox.cpp# LayoutUnit containerMainInnerSize = mainAxisContentExtent(sumHypotheticalMainSize); I am going to mark this as "RESOLVED CONFIGURATION CHANGED". Thanks!