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

[css-align] should be clearer about whether 'align-content' applies to single-line flex containers #1421

Closed
dbaron opened this issue May 20, 2017 · 2 comments

Comments

@dbaron
Copy link
Member

dbaron commented May 20, 2017

In flexbox, it was clear that align-content only applies to multiline flex containers.

But the alignment spec says:

For align-content, the flex lines.

This wording makes it sound like it might do something for single-line flex containers. I tend to think it should be clearer that it doesn't. But if, instead, the intent is that it does, I think it should be much more explicit. (And I'd be a little worried about compatibility.)

@dbaron dbaron added the css-align-3 Current Work label May 20, 2017
@fantasai
Copy link
Collaborator

Fixed in e1ddeea, thanks!

@fantasai
Copy link
Collaborator

(I think it's a bit unfortunate that it doesn't apply to single-line flex containers, but this at least makes the specs more clearly consistent.)

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

2 participants