List item bullet is missing with overflow: hidden
Reported by
stefan.neculai@gmail.com,
Jul 7 2016
|
|||||
Issue descriptionUserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.103 Safari/537.36 Example URL: https://jsfiddle.net/cvwq3td5/ Steps to reproduce the problem: Visit https://jsfiddle.net/cvwq3td5/. What is the expected behavior? What went wrong? The list item bullet is missing. This problem appears to be related to the overflow properties set for the inner elements. Does it occur on multiple sites: Yes Is it a problem with a plugin? No Did this work before? N/A Does this work in other browsers? Yes Chrome version: 51.0.2704.103 Channel: n/a OS Version: OS X 10.11.5 Flash Version: Shockwave Flash 22.0 r0
,
Jul 7 2016
,
Jul 12 2016
While incorrect this is not entirely unexpected given that the default style for list items is display
,
Jul 12 2016
,
Jul 17 2016
I believe the problem must be https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/layout/LayoutListItem.cpp?q=layoutlistitem&sq=package:chromium&dr=CSs&l=400 I just set the adjustOverflow to true. Then the list item marker appears. Can someone help me how to figure out the root cause
,
Mar 24 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/fd1e60f0bee4b6b25d898f4f4f859aa38d1a9643 commit fd1e60f0bee4b6b25d898f4f4f859aa38d1a9643 Author: cathiechen <cathiechen@tencent.com> Date: Fri Mar 24 06:28:02 2017 Get markers back while list items have overflow:hidden children Add marker into list item not overflow:hidden child while construct layout tree. BUG= 626293 Review-Url: https://codereview.chromium.org/2759883002 Cr-Commit-Position: refs/heads/master@{#459372} [add] https://crrev.com/fd1e60f0bee4b6b25d898f4f4f859aa38d1a9643/third_party/WebKit/LayoutTests/fast/lists/list-marker-before-overflow-hidden-expected.html [add] https://crrev.com/fd1e60f0bee4b6b25d898f4f4f859aa38d1a9643/third_party/WebKit/LayoutTests/fast/lists/list-marker-before-overflow-hidden.html [modify] https://crrev.com/fd1e60f0bee4b6b25d898f4f4f859aa38d1a9643/third_party/WebKit/Source/core/layout/LayoutListItem.cpp
,
Nov 9 2017
#6 fixed this problem, thank you, cathiechen@tencent.com!
,
Nov 10 2017
My pleasure:) |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by stefan.neculai@gmail.com
, Jul 7 2016