23053 2008-12-31 00:42:18 -0800 LI element loses its bullet in multi-column mode 2023-10-01 15:46:22 -0700 1 1 1 Unclassified WebKit CSS 528+ (Nightly build) Mac (PowerPC) OS X 10.4 RESOLVED CONFIGURATION CHANGED http://zanstra.com/x/webkit/li-colcount.html P2 Normal --- 1 doekman gur.trio ahmad.saleem792 ap commit-queue craiga darin dbates esprehn+autocc glenn gur.trio hyatt info jackalmage koivisto kondapallykalyan laszlo.gombos mitz robin simon.fraser tabatkins tonikitoo webkit.org yutak zalan oldest_to_newest 103877 0 doekman 2008-12-31 00:42:18 -0800 When setting the CSS property -webkit-column-count, the bullets of the LI-elements disappear. See test-case provided (URL). Also this script: <ul style="-webkit-column-count:2"> <li>One<li>Two<li>Three<li>Four </ul> 237481 1 webkit.org 2010-06-12 22:02:35 -0700 also ordered list numerals, and according to Bug 38835 list-style-image images too https://bugs.webkit.org/show_bug.cgi?id=38835 237483 2 webkit.org 2010-06-12 22:08:02 -0700 Workaround: list-style-position: inside; (Webkit default is presumably outside) 356664 3 ap 2011-02-23 21:28:08 -0800 Still reproducible with r79080. 976715 4 gur.trio 2014-02-05 05:13:24 -0800 (In reply to comment #3) > Still reproducible with r79080. As per the code what is happening is that when we paint the markers it checks whether the marker position intersects with the paint area rect and it fails in that check and does not draw anything. So one fix is when we query the marker property for position (inside or outside) we can return true if UL has columns. Please suggest? 981764 5 gur.trio 2014-02-17 20:16:25 -0800 *** Bug 38835 has been marked as a duplicate of this bug. *** 981788 6 224470 gur.trio 2014-02-17 22:01:06 -0800 Created attachment 224470 Patch 981834 7 gur.trio 2014-02-18 01:15:55 -0800 (In reply to comment #6) > Created an attachment (id=224470) [details] > Patch Please review. Thanks. 984754 8 224470 zalan 2014-02-26 08:53:51 -0800 Comment on attachment 224470 Patch and now even OUTSIDE is treated like INSIDE when <ul> has columns? Not sure if that's correct. It'd be great to figure out why the marker is mispositioned as opposed to force it to behave like INSIDE. 984758 9 224470 simon.fraser 2014-02-26 09:04:11 -0800 Comment on attachment 224470 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=224470&action=review > Source/WebCore/rendering/RenderListMarker.cpp:1741 > - return m_listItem.notInList() || style().listStylePosition() == INSIDE; > + return m_listItem.notInList() || style().listStylePosition() == INSIDE || m_listItem.parent()->hasColumns(); This doesn't seem like the correct way to fix this. 1966679 10 ahmad.saleem792 2023-07-13 16:39:55 -0700 Changed Comment 0 test case in following fiddle: https://jsfiddle.net/fgax1to4/ Safari 16.5.1 is matching with Chrome Canary 117. 1966811 11 darin 2023-07-14 14:59:23 -0700 It’s good that Safari and Chrome match. It’s possible this is just a bug in WebKit that Chrome never tackled though, and we are just comparing WebKit with itself. Ideally we’d also like to know if this behavior matches the CSS and HTML specification intent and whether it matches Firefox. Also, we’d like WPT test coverage to help flag any future changes in this behavior in any of the browsers. We can probably close this bug report without all of that, but that seems like what it would take to thoroughly dispatch this issue. We don’t need WebKit specialists to do any of those things, by the way. That would only be needed if we decided to change WebKit behavior. 1966815 12 jackalmage 2023-07-14 15:17:23 -0700 As far as I can tell, Chrome's behavior is to draw the bullets normally. They're not specially clipped by the column container (as they shouldn't be). So if Safari is matching that and drawing the bullets in the example given here, then it's correct. 1981641 13 ahmad.saleem792 2023-10-01 15:46:22 -0700 All browsers are matching Chrome Canary 119, Firefox Nightly 120 and Safari Technology Preview 179. NOTE: I removed -webkit- prefix from JSFiddle to test on Firefox. Marking this as 'RESOLVED CONFIGURATION CHANGED', please reopen if it is still reproducible. 224470 2014-02-17 22:01:06 -0800 2014-02-26 09:04:11 -0800 Patch bug-23053-20140218113129.patch text/plain 4220 gur.trio SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE2NDI3MikKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDIyIEBACisyMDE0LTAyLTE3ICBHdXJwcmVl dCBLYXVyICA8ay5ndXJwcmVldEBzYW1zdW5nLmNvbT4KKworICAgICAgICBMSSBlbGVtZW50IGxv c2VzIGl0cyBidWxsZXQgaW4gbXVsdGktY29sdW1uIG1vZGUKKyAgICAgICAgaHR0cHM6Ly9idWdz LndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIzMDUzCisKKyAgICAgICAgUmV2aWV3ZWQgYnkg Tk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVGhlIGxpc3QgaXMgYnJva2VuIHVwIGludG8gY29s dW1ucyB1c2luZyAtd2Via2l0LWNvbHVtbi1jb3VudC4gVGhlIExJCisgICAgICAgIGVsZW1lbnRz IGNvbnRlbnRzIGFyZSByZW5kZXJlZCBwcm9wZXJseSBpbiB0d28gY29sdW1ucyBidXQgdGhlIGJ1 bGxldHMKKyAgICAgICAgYXJlIG1pc3NpbmcuIFdoaWxlIHBhaW50aW5nIHRoZSBtYXJrZXJzIGlu IFJlbmRlckxpc3RNYXJrZXI6OnBhaW50LAorICAgICAgICB0aGUgbWFya2VyIHBvc2l0aW9uIGRv ZXNub3QgaW50ZXJzZWN0IHdpdGggcGFpbnRJbmZvLnJlY3QgYW5kIGhlbmNlIGl0cworICAgICAg ICBub3QgZHJhd24uCisKKyAgICAgICAgVGVzdDogZmFzdC9saXN0cy9saXN0LW1hcmtlci1tdWx0 aS1jb2x1bW4uaHRtbAorCisgICAgICAgICogcmVuZGVyaW5nL1JlbmRlckxpc3RNYXJrZXIuY3Bw OgorICAgICAgICAoV2ViQ29yZTo6UmVuZGVyTGlzdE1hcmtlcjo6aXNJbnNpZGUpOgorICAgICAg ICBJbmNhc2Ugb2YgTEkncyBwYXJlbnQgaGFzIGNvbHVtbnMgdGhlIGxpc3QgbWFya2VyIGlzIGRy YXduIGluc2lkZS4KKwogMjAxNC0wMi0xNyAgUmlja3kgTW9uZGVsbG8gIDxybW9uZGVsbG9AYXBw bGUuY29tPgogCiAgICAgICAgIEV4cG9zZSBhIHdheSB0byBjbGVhciBjb29raWVzIG1vZGlmaWVk IGFmdGVyIGEgZ2l2ZW4gZGF0ZQpJbmRleDogU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRl ckxpc3RNYXJrZXIuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL3JlbmRlcmluZy9S ZW5kZXJMaXN0TWFya2VyLmNwcAkocmV2aXNpb24gMTY0MDk0KQorKysgU291cmNlL1dlYkNvcmUv cmVuZGVyaW5nL1JlbmRlckxpc3RNYXJrZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xNzM4LDcg KzE3MzgsNyBAQCBTdHJpbmcgUmVuZGVyTGlzdE1hcmtlcjo6c3VmZml4KCkgY29uc3QKIAogYm9v bCBSZW5kZXJMaXN0TWFya2VyOjppc0luc2lkZSgpIGNvbnN0CiB7Ci0gICAgcmV0dXJuIG1fbGlz dEl0ZW0ubm90SW5MaXN0KCkgfHwgc3R5bGUoKS5saXN0U3R5bGVQb3NpdGlvbigpID09IElOU0lE RTsKKyAgICByZXR1cm4gbV9saXN0SXRlbS5ub3RJbkxpc3QoKSB8fCBzdHlsZSgpLmxpc3RTdHls ZVBvc2l0aW9uKCkgPT0gSU5TSURFIHx8IG1fbGlzdEl0ZW0ucGFyZW50KCktPmhhc0NvbHVtbnMo KTsKIH0KIAogSW50UmVjdCBSZW5kZXJMaXN0TWFya2VyOjpnZXRSZWxhdGl2ZU1hcmtlclJlY3Qo KQpJbmRleDogTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3Rz L0NoYW5nZUxvZwkocmV2aXNpb24gMTY0MjcyKQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3 b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTQtMDItMTcgIEd1cnByZWV0IEthdXIg IDxrLmd1cnByZWV0QHNhbXN1bmcuY29tPgorCisgICAgICAgIExJIGVsZW1lbnQgbG9zZXMgaXRz IGJ1bGxldCBpbiBtdWx0aS1jb2x1bW4gbW9kZQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0 Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjMwNTMKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkg KE9PUFMhKS4KKworICAgICAgICAqIGZhc3QvbGlzdHMvbGlzdC1tYXJrZXItbXVsdGktY29sdW1u LWV4cGVjdGVkLmh0bWw6IEFkZGVkLgorICAgICAgICAqIGZhc3QvbGlzdHMvbGlzdC1tYXJrZXIt bXVsdGktY29sdW1uLmh0bWw6IEFkZGVkLgorICAgICAgICBBZGRlZCBuZXcgdGVzdCBjYXNlIHRv IHRlc3QgdGhlIGZ1bmN0aW9uYWxpdHkgb2YgbXVsdGktY29sdW1uIGxpc3RzLgorCiAyMDE0LTAy LTE3ICBCZW5qYW1pbiBQb3VsYWluICA8YnBvdWxhaW5AYXBwbGUuY29tPgogCiAgICAgICAgIFNl bGVjdG9yQ29tcGlsZXIgaW5jb3JyZWN0bHkgc2F2ZXMgYSBiYWNrdHJhY2tpbmcgcmVnaXN0ZXIg Zm9yIGEgY2hpbGQgY2hhaW4gd2l0aG91dCBkZXNjZW5kYW50IHJlbGF0aW9uIG9uIHRoZSByaWdo dApJbmRleDogTGF5b3V0VGVzdHMvZmFzdC9saXN0cy9saXN0LW1hcmtlci1tdWx0aS1jb2x1bW4t ZXhwZWN0ZWQuaHRtbAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9mYXN0L2xpc3RzL2xpc3Qt bWFya2VyLW11bHRpLWNvbHVtbi1leHBlY3RlZC5odG1sCShyZXZpc2lvbiAwKQorKysgTGF5b3V0 VGVzdHMvZmFzdC9saXN0cy9saXN0LW1hcmtlci1tdWx0aS1jb2x1bW4tZXhwZWN0ZWQuaHRtbAko d29ya2luZyBjb3B5KQpAQCAtMCwwICsxLDEwIEBACis8IURPQ1RZUEUgaHRtbD4KKzxodG1sPgor ICAgIDxib2R5PgorICAgICAgICA8cD5UZXN0cyB0aGF0IHRoZSBidWxsZXRzIGFyZSBub3QgbWlz c2luZy48L3A+CisgICAgICAgIDx1bCBzdHlsZT0iLXdlYmtpdC1jb2x1bW4tY291bnQ6MjtsaXN0 LXN0eWxlLXBvc2l0aW9uOiBpbnNpZGU7Ij4KKyAgICAgICAgICAgIDxsaT5GaXJzdCBDb2x1bW48 L2xpPgorICAgICAgICAgICAgPGxpPlNlY29uZCBDb2x1bW48L2xpPgorICAgICAgICA8L3VsPgor ICAgIDwvYm9keT4KKzwvaHRtbD4KClByb3BlcnR5IGNoYW5nZXMgb246IExheW91dFRlc3RzL2Zh c3QvbGlzdHMvbGlzdC1tYXJrZXItbXVsdGktY29sdW1uLWV4cGVjdGVkLmh0bWwKX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f XwpBZGRlZDogc3ZuOmVvbC1zdHlsZQojIyAtMCwwICsxICMjCituYXRpdmUKXCBObyBuZXdsaW5l IGF0IGVuZCBvZiBwcm9wZXJ0eQpBZGRlZDogc3ZuOmV4ZWN1dGFibGUKIyMgLTAsMCArMSAjIwor KgpcIE5vIG5ld2xpbmUgYXQgZW5kIG9mIHByb3BlcnR5CkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0 L2xpc3RzL2xpc3QtbWFya2VyLW11bHRpLWNvbHVtbi5odG1sCj09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91 dFRlc3RzL2Zhc3QvbGlzdHMvbGlzdC1tYXJrZXItbXVsdGktY29sdW1uLmh0bWwJKHJldmlzaW9u IDApCisrKyBMYXlvdXRUZXN0cy9mYXN0L2xpc3RzL2xpc3QtbWFya2VyLW11bHRpLWNvbHVtbi5o dG1sCSh3b3JraW5nIGNvcHkpCkBAIC0wLDAgKzEsMTAgQEAKKzwhRE9DVFlQRSBodG1sPgorPGh0 bWw+CisgICAgPGJvZHk+CisgICAgICAgIDxwPlRlc3RzIHRoYXQgdGhlIGJ1bGxldHMgYXJlIG5v dCBtaXNzaW5nLjwvcD4KKyAgICAgICAgPHVsIHN0eWxlPSItd2Via2l0LWNvbHVtbi1jb3VudDoy OyI+CisgICAgICAgICAgICA8bGk+Rmlyc3QgQ29sdW1uPC9saT4KKyAgICAgICAgICAgIDxsaT5T ZWNvbmQgQ29sdW1uPC9saT4KKyAgICAgICAgPC91bD4KKyAgICA8L2JvZHk+Cis8L2h0bWw+CgpQ cm9wZXJ0eSBjaGFuZ2VzIG9uOiBMYXlvdXRUZXN0cy9mYXN0L2xpc3RzL2xpc3QtbWFya2VyLW11 bHRpLWNvbHVtbi5odG1sCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX18KQWRkZWQ6IHN2bjpleGVjdXRhYmxlCiMjIC0wLDAg KzEgIyMKKyoKXCBObyBuZXdsaW5lIGF0IGVuZCBvZiBwcm9wZXJ0eQpBZGRlZDogc3ZuOmVvbC1z dHlsZQojIyAtMCwwICsxICMjCituYXRpdmUKXCBObyBuZXdsaW5lIGF0IGVuZCBvZiBwcm9wZXJ0 eQo=