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-page-floats-3][css-box-3] Move definitions of float and clear #7350

Closed
cdoublev opened this issue Jun 13, 2022 · 1 comment
Closed

[css-page-floats-3][css-box-3] Move definitions of float and clear #7350

cdoublev opened this issue Jun 13, 2022 · 1 comment

Comments

@cdoublev
Copy link
Collaborator

cdoublev commented Jun 13, 2022

both, both-inline, both-block, are not included in the value of the definition table for the clear property in CSS Page Floats 3, which is currently the primary value definition for clear, but are defined in prose below the clear definition table.

EDIT

According to this comment, clear and float should be defined in CSS Box 3 (or a module for block layout) therefore I change my demand by updating the title of this issue, and by asking for the following changes in CSS Page Floats 3 (after defining float and clear in another spec):

  Name: `float`
- Value: `block-start | block-end | inline-start | inline-end | snap-block | <snap-block()> | snap-inline | <snap-inline()> | left | right | top | bottom | none`
+ New values: `block-start | block-end | inline-start | inline-end | snap-block | <snap-block()> | snap-inline | <snap-inline()> | top | bottom`

  Name: `clear`
- Value: `inline-start | inline-end | block-start | block-end | left | right | top | bottom | none`
+ New values: `inline-start | inline-end | block-start | block-end | top | bottom`
@cdoublev cdoublev changed the title [css-page-floats-3] Missing keywords in the value definition of the clear property [css-page-floats-3][css-box-3] Move definitions of float and clear Oct 7, 2022
@fantasai
Copy link
Collaborator

Fixed in #9105

CSS Block Layout Level 3 is still a pipe dream, so closing this out for now. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants