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

Number syntax: digit-sequence "." #442

Closed
dirkschulze opened this issue May 10, 2018 · 6 comments
Closed

Number syntax: digit-sequence "." #442

dirkschulze opened this issue May 10, 2018 · 6 comments

Comments

@dirkschulze
Copy link
Contributor

SVG Path data as well as SVG transforms and probably other SVG specific attributes that take numbers allow: 1..

It seems only Edge does support this:
https://codepen.io/krit/pen/wjmzzQ
https://codepen.io/krit/pen/rvdOQX

Should this be removed? Do we just need more tests?

Note: CSS numbers do not support this syntax either.

https://www.w3.org/TR/SVG/paths.html#PathDataBNF
https://www.w3.org/TR/SVG/coords.html#TransformAttribute (replaced by CSS Transforms but sill relevant since the number syntax should be defined the same for all appearances. w3c/csswg-drafts#2623 (comment))

@dirkschulze
Copy link
Contributor Author

Adobe Illustrator does support this syntax as well.

@dstorey @AmeliaBR @longsonr could you chime in please and say what you think about this?

@dirkschulze
Copy link
Contributor Author

As an interesting note: The x, y, width and height attributes do support the digit-sequence "." syntax in Safari, Chrome and Edge but not in Firefox.

https://codepen.io/krit/pen/JvLrbv

@AmeliaBR
Copy link
Contributor

This is a duplicate of #331.

Could you add your comments there, Dirk?

@dirkschulze
Copy link
Contributor Author

@AmeliaBR Thanks. I did create and added a summary there.

@longsonr
Copy link

Number parsing should be consistent with CSS rules. I highly doubt Firefox will change unless the CSS spec changes.

@dirkschulze
Copy link
Contributor Author

@longsonr as @AmeliaBR said, comment on #331 please.

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

3 participants