Skip to content

[css-contain] property names linkified#2801

Merged
tabatkins merged 1 commit into
w3c:masterfrom
ewilligers:contain-linkify-property-names
Jun 21, 2018
Merged

[css-contain] property names linkified#2801
tabatkins merged 1 commit into
w3c:masterfrom
ewilligers:contain-linkify-property-names

Conversation

@ewilligers
Copy link
Copy Markdown
Contributor

'counter-increment' and 'content' property names are now working links.

resolves #2800

'counter-increment' and 'content' property names are now working links.

resolves w3c#2800
@tabatkins tabatkins merged commit 7868271 into w3c:master Jun 21, 2018
@ewilligers ewilligers deleted the contain-linkify-property-names branch June 21, 2018 20:18
@frivoal
Copy link
Copy Markdown
Collaborator

frivoal commented Jun 22, 2018

Sorry, this needs to be reverted:

  1. linking to css-content instead of css2 is not appropriate. css-content is a known-to-be buggy draft. It is not as bad as it used to be, as some maintenance work has been done to it, but I don't think it has reached the point where it is more accurate than css2. Case in point: it claims that the content property applies to all elements, which is absolutely not true in existing implementations. css2 correctly says that it applies to ":before and :after pseudo-elements "

  2. independently of the point above, neither css-content nor css-list-3 are mature enough to be accepted without strong justifications as normative references for a document which is trying to exit CR, so we will have to change these links anyway before next publication of css-contain

@frivoal
Copy link
Copy Markdown
Collaborator

frivoal commented Jun 22, 2018

sub-point on css-list-3: for counter-set, css-list-3 is vaguely acceptable, as:

  1. counter-set is not defined anywhere else
  2. it can be argued that counter-set is almost identical to counter-reset, which is in a REC.

That makes it possible to argue in favor of using css-list-3 as a reference for counter-set, but harder to argue in favor of it for things that exist in CSS2.

frivoal added a commit that referenced this pull request Jun 22, 2018
The newer specs are very unstable (and marked as such), and hard to
justify as normative dependencies for TR publications for a CR-or-later
spec.

Fixes #2800
Replaces #2801
@frivoal
Copy link
Copy Markdown
Collaborator

frivoal commented Jun 22, 2018

Commit 26d5bf6 fixes this. And since it uses css2 rather than css21, we do get cross-spec links.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[css-contain] 'counter-increment' and 'content' property names should be linkified (but are not), in style containment section

4 participants