Skip to content

Conversation

@SebastianZ
Copy link
Contributor

No description provided.

@frivoal
Copy link
Collaborator

frivoal commented Jun 21, 2016

The multicol part of this patch looks good. For other specs, see with their respective editors.

@SebastianZ
Copy link
Contributor Author

SebastianZ commented Jun 22, 2016

Thank you for the review, Florian! Should I do something to get a review from the other editors?

Sebastian


<h2 id="logical-controls">
Logical Directional Values: ''block-start'', ''block-end'', ''inline-start'', ''inline-end''</h2>
Logical Directional Values: 'block-start', 'block-end', 'inline-start', 'inline-end'</h2>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These (and the next) are wrong - 'foo' is the shorthand for property autolinks, ''foo'' is for values, and is being used correctly.

@tabatkins
Copy link
Member

I reviewed all of them. Left some comments; the rest are good.

@tabatkins tabatkins merged commit 71e4ee9 into w3c:master Jun 22, 2016
@SebastianZ SebastianZ deleted the issue-208 branch June 22, 2016 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants