Floats, negative margins and pseudo elements don't play well together
Reported by
bbri...@microsoft.com,
Mar 2 2016
|
||||
Issue descriptionReduced problem: https://jsfiddle.net/8f4jj1ja/ Edge and Firefox has matching rendering meeting the expectation. Blink produces different result Found during compat testing on live site: http://www.hrej.cz/ (look at the right section with "Přihlášení" title and observe the social icons). In all honesty very low severity and priority issue - but filing for completeness.
,
Mar 5 2016
The second span should avoid the float. It does if there's whitespace between the two spans.
,
Mar 6 2016
,
Mar 9 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/24bc38bac8c8f2ca1ee61b8872829fcdde521fa3 commit 24bc38bac8c8f2ca1ee61b8872829fcdde521fa3 Author: robhogan <robhogan@gmail.com> Date: Wed Mar 09 22:51:57 2016 Clear floats even when they are not tracked by a previous sibling BUG= 591243 Review URL: https://codereview.chromium.org/1766223002 Cr-Commit-Position: refs/heads/master@{#380248} [add] https://crrev.com/24bc38bac8c8f2ca1ee61b8872829fcdde521fa3/third_party/WebKit/LayoutTests/fast/block/float/block-with-negative-margin-clears-float-expected.txt [add] https://crrev.com/24bc38bac8c8f2ca1ee61b8872829fcdde521fa3/third_party/WebKit/LayoutTests/fast/block/float/block-with-negative-margin-clears-float.html [modify] https://crrev.com/24bc38bac8c8f2ca1ee61b8872829fcdde521fa3/third_party/WebKit/Source/core/layout/LayoutBlockFlow.cpp
,
Mar 18 2016
,
Mar 18 2016
Nice!! Thank you so much for fixing this - verified on repro and live site. I'll be following up with the outreach to let the live site know about this change. |
||||
►
Sign in to add a comment |
||||
Comment 1 by e...@chromium.org
, Mar 4 2016Status: Available (was: Untriaged)