Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error Processing should talk about when to use initial values for attributes #52

Closed
heycam opened this issue Feb 24, 2016 · 4 comments
Closed

Comments

@heycam
Copy link
Contributor

heycam commented Feb 24, 2016

The current https://svgwg.org/svg2-draft/implnote.html#ErrorProcessing section doesn't refer to initial/invalid values of attributes, so it's not exactly clear that you're meant to use them when an attribute is invalid in some way. For example looking at https://svgwg.org/svg2-draft/coords.html#ViewBoxAttribute it says that negative values for width/height inside the viewBox are an error, but the Error Processing section linked to doesn't make it clear that the behaviour should be as if the attribute weren't specified (as that's what's specified for the "initial value" in the grey attribute definition box).

@sideshowbarker
Copy link
Contributor

See https://www.khronos.org/registry/vulkan/specs/1.0/xhtml/images/tstripadj.svg for a case where this results in different behavior between Chrome and Firefox due to differences in what they do when they run into a (invalid) negative viewBox value.

@philiptaylor
Copy link

@nikosandronikos
Copy link
Member

3.2. Attribute syntax does note the following

The Initial value column gives the initial value for the attribute. When an attribute fails to parse according to the specified CSS Value Definition Syntax, ABNF or EBNF grammar, or if parsing according to the URL Standard or by the prose describing how to parse the attribute indicates failure, the attribute is assumed to have been specified as the given initial value.

So I think we should just remove the link to 'Error Processing'.

@nikosandronikos
Copy link
Member

We might also just want to check that viewBox is clear about throwing the whole viewBox away if one parameter isn't correct.

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

No branches or pull requests

4 participants