Bug 163752 - Update <ol>.start getter to default to 1
Summary: Update <ol>.start getter to default to 1
Status: RESOLVED DUPLICATE of bug 178057
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-10-20 12:55 PDT by Domenic Denicola
Modified: 2017-10-07 19:03 PDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Domenic Denicola 2016-10-20 12:55:19 PDT
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
Comment 1 Maxime Plante 2016-10-20 20:29:38 PDT
Hi! I'm a newcomer here.

It seems pretty simple to solve. I am working on it.
Comment 2 Chris Dumez 2016-10-20 20:30:45 PDT
(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.
Comment 3 Chris Dumez 2016-10-20 20:32:00 PDT
(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.
Comment 4 Chris Dumez 2016-10-20 20:33:03 PDT
(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.
Comment 5 Maxime Plante 2016-10-20 20:46:24 PDT
Wait and see then. Thanks for the help!
Comment 6 Chris Dumez 2017-10-07 19:03:35 PDT

*** This bug has been marked as a duplicate of bug 178057 ***