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

letter-spacing definition "removes" percentage - not possible #611

Closed
dirkschulze opened this issue Dec 8, 2018 · 4 comments
Closed

letter-spacing definition "removes" percentage - not possible #611

dirkschulze opened this issue Dec 8, 2018 · 4 comments

Comments

@dirkschulze
Copy link
Contributor

The SVG2 spec says that percentage get "removed" from the letter-spacing property. That is not possible. Just like for word-spacing we need to decide what percentage values resolve to or are relative to.

https://svgwg.org/svg2-draft/text.html#LetterSpacingProperty

@dirkschulze
Copy link
Contributor Author

Ideally, all definitions should move to the responsible specifications. Even the SVG specific bits I think.

@AmeliaBR
Copy link
Contributor

It's removed in the sense that it is not a valid value, per CSS Text Level 3.

This is a change from the definition in SVG 1.1, which allowed percentages as SVG lengths.

I haven't tested whether any browsers support percentage values as defined in SVG 1.1, but I doubt it. If they did, that would be an issue to take up with CSS to make the spec match reality.

The statement about the change should really be part of a note, rather than a normative statement, though.

@dirkschulze
Copy link
Contributor Author

@AmeliaBR Ah SVG 1.1 at its best. Thanks for the clarification. In a short test, Safari, Firefox and Chrome do not support percentage.

@svgeesus
Copy link
Contributor

svgeesus commented Jan 7, 2019

So instead of "removes" we should say "no longer allows" percentage values relative to the size of viewport.

svgeesus added a commit that referenced this issue Jan 7, 2019
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