Skip to content

[css-align-3] Add left|right to *-position grammar #2169

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

Closed

Conversation

sideshowbarker
Copy link
Member

This change adds the left and right keywords to the grammar terms
<self-position> and <content-position>.

That addition to the grammar brings it into sync with the existing spec prose,
which already includes left and right among the positional alignment
keywords collection, and states, “Two grammar terms are used to denote this
collection of values: <self-position> [and] <content-position>”.

This change adds the `left` and `right` keywords to the grammar terms
`<self-position>` and `<content-position>`.

That addition to the grammar brings it into sync with the existing spec prose,
which already includes `left` and `right` among the *positional alignment*
keywords collection, and states, “Two grammar terms are used to denote this
collection of values: `<self-position>` [and] `<content-position>`”.
@sideshowbarker sideshowbarker added the css-align-3 Current Work label Jan 6, 2018
@tabatkins
Copy link
Member

Yeah, I talked with @fantasai about this and she reminded me that we very explicitly did not want left/right in justify-content, nor the align-* properties; they're only intended for inline layout.

@tabatkins tabatkins closed this Jan 8, 2018
@sideshowbarker sideshowbarker deleted the sideshowbarker/position-grammar-add-left-right branch January 30, 2018 03:48
@syncbot syncbot restored the sideshowbarker/position-grammar-add-left-right branch February 6, 2018 19:39
@plinss plinss deleted the sideshowbarker/position-grammar-add-left-right branch February 16, 2018 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
css-align-3 Current Work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants