Bug 172446

Summary:

Ordered list and Shadow DOM have broken order

Product:

WebKit

Reporter:

Mattia Astorino <astorino.design>

Component:

DOM

Assignee:

Nobody <webkit-unassigned>

Status:

NEW

Severity:

Major

CC:

cdumez, changseok, esprehn+autocc, ews-watchlist, glenn, kangil.han, koivisto, kondapallykalyan, pdr, rniwa, webkit-bug-importer, zalan

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Mac

OS:

macOS 10.12

Bug Depends on:

Bug Blocks:

148695

Attachments:

Description	Flags
Reproduction	none
WIP	ews-feeder: commit-queue-
WIP2	none

Mattia Astorino

Reported 2017-05-22 06:41:30 PDT

When an ordered list is composed by custom elements and shadow dom (open and slot) the list counter restart each <li> element, so every <li> have the 1. number: This is the output: 1. Lorem ipsum 1. Dolor Sit amet 1. Consecutor adipiscing 1. Lorem ipsum 1. Dolor Sit amet 1. Dolor Sit amet This is the source code: <oa-list> <oa-list-item>Lorem ipsum</oa-list-item> <oa-list-item>Dolor Sit amet</oa-list-item> <oa-list-item> Consecutor adipiscing <oa-list> <oa-list-item>Lorem ipsum</oa-list-item> <oa-list-item>Dolor Sit amet</oa-list-item> </oa-list> </oa-list-item> <oa-list-item>Dolor Sit amet</oa-list-item> </oa-list> <oa-list> shadow root: <ol> <slot></slot> </ol> <oa-list-item> shadow root: <li> <slot></slot> </li>

Attachments
Reproduction (915 bytes, text/html) 2018-09-18 01:06 PDT, Ryosuke Niwa	no flags	Details
WIP (8.73 KB, patch) 2022-07-17 14:48 PDT, Ryosuke Niwa	ews-feeder: commit-queue-	Details Formatted Diff Diff
WIP2 (8.83 KB, patch) 2022-07-17 23:50 PDT, Ryosuke Niwa	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Mattia Astorino

Comment 1 2017-06-06 00:20:31 PDT

Now with the latest WebKit Nightly release the there is no bullet indicators for orderered and unordered lists. Are you working on that? This is the output, same source code: http://i.imgur.com/aEaaMBc.png

Antti Koivisto

Comment 2 2017-11-03 05:19:58 PDT

A full test case would be helpful.

Radar WebKit Bug Importer

Comment 3 2018-08-01 22:40:47 PDT

<rdar://problem/42842990>

Ryosuke Niwa

Comment 4 2018-09-18 01:06:26 PDT

Created attachment 350010 [details] Reproduction This works in Chrome & Firefox but fails in Safari.

Ryosuke Niwa

Comment 5 2022-07-17 14:48:00 PDT

Created attachment 460965 [details] WIP

Ryosuke Niwa

Comment 6 2022-07-17 23:50:20 PDT

Created attachment 460967 [details] WIP2

Ryosuke Niwa

Comment 7 2022-07-18 14:23:14 PDT

Hm... attached patch fixes the order number issue but there is some kind of weird padding / margin added to the list. Not sure where that's coming from.

Note You need to log in before you can comment on or make changes to this bug.