Bug 163752
| Summary: | Update <ol>.start getter to default to 1 | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Domenic Denicola <d> |
| Component: | DOM | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED DUPLICATE | ||
| Severity: | Normal | CC: | cdumez, maxime.pl |
| Priority: | P2 | ||
| Version: | Safari Technology Preview | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Domenic Denicola
Currently in Edge and Gecko, <ol>.start defaults to 1. In Blink and WebKit, it defaults to something that depends on how many children are rendered. In https://github.com/whatwg/html/pull/1916 we updated the spec to follow the simpler Firefox/Edge behavior, with cdumez's cautious support.
Tests are available at http://w3c-test.org/html/semantics/grouping-content/the-ol-element/grouping-ol.html
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Maxime Plante
Hi! I'm a newcomer here.
It seems pretty simple to solve. I am working on it.
Chris Dumez
(In reply to comment #1)
> Hi! I'm a newcomer here.
>
> It seems pretty simple to solve. I am working on it.
I discourage you from starting with this. This is a risky change as Blink is currently adding a UserCounter to determine how risky it is.
Chris Dumez
(In reply to comment #2)
> (In reply to comment #1)
> > Hi! I'm a newcomer here.
> >
> > It seems pretty simple to solve. I am working on it.
>
> I discourage you from starting with this. This is a risky change as Blink is
> currently adding a UserCounter to determine how risky it is.
In English:
I discourage you from starting with this. This is a risky change and Blink is
currently adding a UseCounter to determine how risky it is. I don't think we should make this change until Blink usage has shown it seems acceptable.
Chris Dumez
(In reply to comment #3)
> (In reply to comment #2)
> > (In reply to comment #1)
> > > Hi! I'm a newcomer here.
> > >
> > > It seems pretty simple to solve. I am working on it.
> >
> > I discourage you from starting with this. This is a risky change as Blink is
> > currently adding a UserCounter to determine how risky it is.
>
> In English:
> I discourage you from starting with this. This is a risky change and Blink is
> currently adding a UseCounter to determine how risky it is. I don't think we
> should make this change until Blink usage has shown it seems acceptable.
See Blink bug at https://bugs.chromium.org/p/chromium/issues/detail?id=657972#c1.
Maxime Plante
Wait and see then. Thanks for the help!
Chris Dumez
*** This bug has been marked as a duplicate of bug 178057 ***