147795 2015-08-07 15:26:01 -0700 Error handling padding in box-sizing calculations 2022-12-21 13:49:08 -0800 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Unspecified Unspecified RESOLVED CONFIGURATION CHANGED BlinkMergeCandidate, InRadar P2 Normal --- 1 bfulgham zalan ahmad.saleem792 bfulgham ddkilzer hyatt simon.fraser webkit-bug-importer oldest_to_newest 1115901 0 258534 bfulgham 2015-08-07 15:26:01 -0700 Created attachment 258534 Test case showing the improper behavior WebKit seems to be taking padding into account when performing height calculations, even when in standards mode using "box-sizing: content-box". 1117281 1 zalan 2015-08-13 10:54:42 -0700 Blink changed available logical height behavior about a year ago. It returns content box height now (vs. border box height) https://src.chromium.org/viewvc/blink?revision=178693&view=revision @@ -3085,7 +3061,7 @@ LayoutUnit RenderBox::computeReplacedLogicalHeightUsing(Length logicalHeight) co LayoutUnit RenderBox::availableLogicalHeight(AvailableLogicalHeightType heightType) const { - return constrainLogicalHeightByMinMax(availableLogicalHeightUsing(style().logicalHeight(), heightType), -1); + return constrainContentBoxLogicalHeightByMinMax(availableLogicalHeightUsing(style().logicalHeight(), heightType), -1); } 1227795 2 webkit-bug-importer 2016-09-08 14:06:46 -0700 <rdar://problem/28213830> 1909290 3 ahmad.saleem792 2022-10-31 06:23:51 -0700 I am not able to reproduce this bug using Safari 16.1 and the "red" square is center similar to Chrome Canary 109 and Firefox Nightly 108. Do we need to anything else? Thanks! 1920218 4 ahmad.saleem792 2022-12-17 03:33:51 -0800 Also we are passing the test (in STP160) from the Chromium patch of Comment 01: Test Case - https://jsfiddle.net/0ytgLhwn/ Do we need to do anything else? 1921206 5 zalan 2022-12-21 13:49:08 -0800 (In reply to Ahmad Saleem from comment #4) > Also we are passing the test (in STP160) from the Chromium patch of Comment > 01: > > Test Case - https://jsfiddle.net/0ytgLhwn/ > > Do we need to do anything else? Nope, this looks to have progressed. Thanks! 258534 2015-08-07 15:26:01 -0700 2015-08-07 15:26:01 -0700 Test case showing the improper behavior padding.html text/html 657 bfulgham PCFET0NUWVBFIGh0bWw+CjxodG1sIGxhbmc9ImVuIj4KPGhlYWQ+CiAgPG1ldGEgaHR0cC1lcXVp dj0iQ29udGVudC1UeXBlIiBjb250ZW50PSJ0ZXh0L2h0bWw7IGNoYXJzZXQ9dXRmLTgiPgogIDx0 aXRsZT5QYWRkaW5nIGlzc3VlczwvdGl0bGU+CiAgPHN0eWxlIHR5cGU9InRleHQvY3NzIiBtZWRp YT0ic2NyZWVuIj4KICAgICNjb250YWluZXIgewogICAgICAgIHBvc2l0aW9uOiByZWxhdGl2ZTsK ICAgICAgICBoZWlnaHQ6IDYwMHB4OwogICAgICAgIHBhZGRpbmctYm90dG9tOiAxMDAwcHg7CiAg ICAgICAgYm9yZGVyOiAxcHggc29saWQgYmx1ZTsKICAgIH0KCiAgICAjYm94IHsKICAgICAgcG9z aXRpb246IHJlbGF0aXZlOwogICAgICB0b3A6IDUwJTsKICAgICAgaGVpZ2h0OiAxMDBweDsKICAg ICAgd2lkdGg6IDEwMHB4OwogICAgICBiYWNrZ3JvdW5kLWNvbG9yOiByZWQ7CiAgICAgIG1hcmdp bjogMDsKICAgIH0KICA8L3N0eWxlPgo8L2hlYWQ+Cjxib2R5PgogICAgPGRpdj5UaGUgYm94IHNo b3VsZCBiZSBjZW50ZXJlZCBpbiB0aGUgdmlld3BvcnQsIHJlZ2FyZGxlc3Mgb2YgcGFkZGluZy48 L2Rpdj4KICAgIDxkaXYgaWQ9ImNvbnRhaW5lciI+CiAgICAgICAgPGRpdiBpZD0iYm94Ij48L2Rp dj4KICAgIDwvZGl2Pgo8L2JvZHk+CjwvaHRtbD4K