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-text] Bad box terminology for text-wrap #2545

Closed
Loirooriol opened this issue Apr 11, 2018 · 1 comment
Closed

[css-text] Bad box terminology for text-wrap #2545

Loirooriol opened this issue Apr 11, 2018 · 1 comment

Comments

@Loirooriol
Copy link
Contributor

From https://drafts.csswg.org/css-text-4/#text-wrap,

Applies to: all elements

Should probably be "Applies to: inline boxes and block containers"

Same as wrap for inline-level elements.

Should probably be "Same as wrap for inline boxes." in both occurrences

Same as wrap for block-level elements

Should probably be "Same as wrap for block containers"

For block-level elements that contain line boxes as direct children

Should probably be "For block containers that contain line boxes as direct children". But I think line 'boxes' only exist in the fragment tree so they are not really children of a box. So "For block containers that establish an inline formatting context".

Additionally, it seems wrap and nowrap don't really apply to block containers, their effect is only via inheritance to inline boxes. Consider clarifying this.

@tabatkins tabatkins added the css-text-3 Current Work label May 9, 2018
@fantasai fantasai added css-text-4 and removed css-text-3 Current Work labels Jun 28, 2018
@fantasai
Copy link
Collaborator

Cleaned up the wording, thanks! https://drafts.csswg.org/css-text-4/#text-wrap

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