w3c / w3c/csswg-drafts

[css-syntax-3] Potentially missing a "parse error" specifier in "consume a URL token"?

Open
#7,987 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

css-syntax-3
Dominant language
Bikeshed
Stars
4.9k
Forks
816
PR merge metrics
PR metrics pending

Description

  • please tag the issue title with the spec's shortname, like [css-foo]
    (this is the name from the spec URL, without a level number unless the issue is specific to that level).
    If you're proposing a new feature that doesn't obviously fit in an existing spec, skip this part — don't make something up.

[css-syntax-3]

  • please be specific (in the title and issue) about what you want to change:
    “make it better” means different things to different people!

The specification has a paragraph that is as follows:
image

  1. If the next input code point is \u0029, we are successful and return the URL token
  2. If we have reached the end-of-file, we record a parse error and return the URL token
  3. Otherwise, we consume the remnants and return a bad URL token

In the third case, we have obviously run into poor, malformed syntax. However, the spec never actually says that a parse error is given in this case, so a spec-compliant validator would not actually report a problem. This leads me to think that we are missing a parse error.
(Note: there is a parse error defined for the second case, but the way the spec is worded it does not apply to the third case)

  • please link to the spec section you're talking about, or at least the spec

CSS Syntax Module Level 3

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the CSS Syntax Module Level 3 “consume a URL token” section linked in the issue. Compare the three listed branches, especially the bad URL token case, with the specification’s parse-error terminology and surrounding algorithms. Done means the issue’s proposed wording is resolved in the spec or the concern is clearly closed.

Written by the indexing model from the issue text.

Assessment

Tech stack
css
Domain
documentation
Issue type
Documentation
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.