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

Harmonise inline-size with CSS Logical properties #436

Closed
dstorey opened this issue May 5, 2018 · 3 comments
Closed

Harmonise inline-size with CSS Logical properties #436

dstorey opened this issue May 5, 2018 · 3 comments

Comments

@dstorey
Copy link
Member

dstorey commented May 5, 2018

from @AmeliaBR

allowing number as a replacement for length happens automatically in all presentation attributes, see https://svgwg.org/svg2-draft/types.html#syntax. Explicitly allowing number in the CSS grammar allows it in the style property, too (this is needed for many SVG-specific properties from SVG 1). Since this [inline-size] is a new property, there is no legacy reason to allow plain number in the CSS property, it is probably best to harmonize with the definition in Logical Properties.

@boggydigital
Copy link
Contributor

Not blocking updated 2.0 CR publication - assigning 2.0 Recommendation milestone to clean this up before 2.0 REC

@Tavmjong Tavmjong self-assigned this Jul 2, 2018
@css-meeting-bot
Copy link
Member

The Working Group just discussed Harmonize `inline-size` .

The full IRC log of that discussion <AmeliaBR> Topic: Harmonize `inline-size`
<AmeliaBR> GitHub: https://github.com//issues/436
<AmeliaBR> Tav: I could probably do this, not sure what's needed.
<AmeliaBR> Amelia: This was just about removing plain number from the syntax, since there is no legacy need.
<AmeliaBR> Tav: OK, that's reasonable.

@dirkschulze
Copy link
Contributor

Adding needs testing since it is a change to the spec. At least one test for SVG should be added.

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

6 participants