Re: [csswg-drafts] [css-align] "the relevant computed self-alignment property" is unnecessarily vague (it can only be "align-self")

Quoting my suggest:
>  (i.e. that phrase should just be replaced with "its computed align-self", I think)

Actually, this^ isn't quite right either.  The spec **doesn't really** want to check whether the computed `align-self` is `stretch`, because that doesn't account for `align-self:auto` (which behaves as `stretch` under default conditions) and `align-self:normal` (which behaves as `stretch` for some boxes, e.g. flex items).

Perhaps we should keep the sentence hand-wavy, though (e.g. "computed align-self is...stretch"), and then add an explanatory afterthought to say that for the purposes of this check, "stretch" includes other values that are defined to behave as `stretch`, including e.g. `auto` when the parent has `align-items:stretch`...?

-- 
GitHub Notification of comment by dholbert
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/2986#issuecomment-410366140 using your GitHub account

Received on Friday, 3 August 2018 20:22:05 UTC