-
Notifications
You must be signed in to change notification settings - Fork 132
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
dur attribute should permit leading . #394
Comments
For usability & consistency with other parts of SVG, I'd definitely like to support making the leading zero optional. (Although the grammar fix might need to be a little more complicated than you suggest, as it makes all parts of a "timecount" value optional.) That said, this syntax was defined for compatibility with SMIL, exactly duplicating their syntax. I'm not sure whether or not that's still relevant now, but it makes spec'ing a change more complicated. |
We could do this instead Timecount-val ::= (Timecount ("." Fraction)? (Metric)?) | ("." Fraction (Metric)?) |
The Working Group just discussed The full IRC log of that discussion<krit> topic: dur attribute should permit leading<liam> github issue: https://github.com//issues/394 <krit> liam: AmeliaBR points out that SMIL doesn't support the dot <krit> AmeliaBR: can we redefine it in SVG with a new syntax or do we continue to align with SMIL? This is not in SVG 2.0 anymore. <krit> krit: suggest to leave issue for later. <krit> AmeliaBR: we should discuss number issues in general first. (differences between CSS and SVG) |
Not blocking updated 2.0 CR publication - assigning 2.1 WD milestone |
Note begin and any other SMIL attributes that use clock-time are also affected. |
From version 115, Firefox will accept dur/begin/end that starts with a decimal point. |
https://svgwg.org/specs/animations/#ClockValueSyntax
dur=".5s" is invalid per the specification. Chrome allows it, Firefox does not. I propose that it be allowed.
So we'd change to this...
Timecount ::= DIGIT*
The text was updated successfully, but these errors were encountered: