239976 2022-05-02 12:03:37 -0700 WebKit pushes down auto-width BFC alongside float, if the BFC has any negative margin at all 2024-05-23 16:53:37 -0700 1 1 1 Unclassified WebKit Layout and Rendering Safari 15 Unspecified Unspecified NEW https://bugs.webkit.org/show_bug.cgi?id=240128 https://bugs.webkit.org/show_bug.cgi?id=274627 InRadar P2 Normal --- 1 dholbert webkit-unassigned ahmad.saleem792 bfulgham karl+webkit simon.fraser webkit-bug-importer zalan oldest_to_newest 1865906 0 458698 dholbert 2022-05-02 12:03:37 -0700 Created attachment 458698 testcase 1 STR: 1. Load attached testcase. EXPECTED RESULTS: The three sections should all have the same layout. ACTUAL RESULTS: In the lower two sections (the ones with a small negative margin), the aqua BFC gets pushed (i.e. cleared/wrapped) down below the fuchsia float. Firefox and Chrome (Gecko/Blink) give EXPECTED RESULTS. Safari (WebKit) gives ACTUAL RESULTS. Note that there's a lot of subtlety and spec ambiguity in this area. I do think it makes sense for sufficiently-large negative margins to cause content to wrap -- specifically if those margins are negative enough to "prop up" the border-box to be larger than the available space. See https://bugzilla.mozilla.org/show_bug.cgi?id=1745310#c9 for more thoughts on this. (I'm implementing this behavior in Gecko over there.) In this particular attached testcase, though, with -5px margins (or even -1px margins), WebKit wraps even though there's no reason to do so that I can see. And WebKit is alone in this behavior, so I'm tentatively filing this as a WebKit bug. 1867719 1 webkit-bug-importer 2022-05-09 12:04:13 -0700 <rdar://problem/92978014> 1909669 2 ahmad.saleem792 2022-11-01 16:53:52 -0700 @Alan - do you think this will help in above case? https://github.com/WebKit/WebKit/commit/c80dcb41f40298f9d1251e3773cda13fe5f3c910 I think it will resolve this negative margin case and we should just mark it close but I am happy to wait till it ships in Safari Technology Preview and test it. 1909832 3 zalan 2022-11-02 12:41:49 -0700 (In reply to Ahmad Saleem from comment #2) > @Alan - do you think this will help in above case? > > https://github.com/WebKit/WebKit/commit/ > c80dcb41f40298f9d1251e3773cda13fe5f3c910 > > I think it will resolve this negative margin case and we should just mark it > close but I am happy to wait till it ships in Safari Technology Preview and > test it. It still looks broken after 256132@main. 458698 2022-05-02 12:03:37 -0700 2022-05-02 12:03:37 -0700 testcase 1 test.html text/html 719 dholbert PCFET0NUWVBFIGh0bWw+CjxzdHlsZT4KLmNvbnRhaW5lciB7CiAgd2lkdGg6IDQwMHB4OwogIGhl aWdodDogNDBweDsKICBib3JkZXI6IDFweCBzb2xpZCBncmF5Owp9CgouZmxvYXQgewogIGZsb2F0 OiBsZWZ0OwogIHdpZHRoOiA1MHB4OwogIGhlaWdodDogOHB4OwogIGJhY2tncm91bmQ6IGZ1Y2hz aWEgbm9uZSByZXBlYXQgc2Nyb2xsIDAlIDAlOwogIGJvcmRlcjogMXB4IHNvbGlkIHB1cnBsZTsK fQoKLmJmYyB7CiAgZGlzcGxheTogZmxvdy1yb290OwogIGJhY2tncm91bmQ6IGFxdWEgbm9uZSBy ZXBlYXQgc2Nyb2xsIDAlIDAlOwogIGJvcmRlcjogMXB4IHNvbGlkIGJsdWU7Cn0KPC9zdHlsZT4K PGRpdiBjbGFzcz0iY29udGFpbmVyIj4KICA8ZGl2IGNsYXNzPSJmbG9hdCI+PC9kaXY+CiAgPGRp diBjbGFzcz0iYmZjIiBzdHlsZT0ibWFyZ2luLWxlZnQ6IDBweDsiPm5vIG1hcmdpbnM8L2Rpdj4K PC9kaXY+CjxkaXYgY2xhc3M9ImNvbnRhaW5lciI+CiAgPGRpdiBjbGFzcz0iZmxvYXQiPjwvZGl2 PgogIDxkaXYgY2xhc3M9ImJmYyIgc3R5bGU9Im1hcmdpbi1sZWZ0Oi01cHg7Ij5tYXJnaW4tbGVm dDotNXB4PC9kaXY+CjwvZGl2Pgo8ZGl2IGNsYXNzPSJjb250YWluZXIiPgogIDxkaXYgY2xhc3M9 ImZsb2F0Ij48L2Rpdj4KICA8ZGl2IGNsYXNzPSJiZmMiIHN0eWxlPSJtYXJnaW4tcmlnaHQ6LTVw eDsiPm1hcmdpbi1yaWdodDotNXB4PC9kaXY+CjwvZGl2Pgo=