Skip to content

Conversation

@felipecrv
Copy link

No description provided.

@tabatkins
Copy link
Member

This is correcting in the wrong direction - 0 1 auto is the correct initial value.

The actual issue here is that shorthand props don't have initial values, so the 'flex' block is wrong - it should just say "see individual properties", like it does for all the other rows.

@SebastianZ
Copy link
Contributor

The actual issue here is that shorthand props don't have initial values, so the 'flex' block is wrong - it should just say "see individual properties", like it does for all the other rows.

I think that statement is not correct, shorthands may have initial values, they are just normally built up by their related longhand values. While there is no such case I know of, e.g. flex could have had none as initial value.
Also, 'see individual properties' is not very expressive. (Therefore MDN lists the longhands in those cases.) So, listing the individual longhands would be better, though that's out of the scope of this change here.

Sebastian

@dholbert
Copy link
Member

@tabatkins, that's what it used to say, BTW - but the full "initial value" got explicitly added here (by @fantasai), recently:
96708b9#diff-9c5fc491cf3bc3eb549a06890ba15b4fL1482

Sounds like that was a mistake, though?

@tabatkins
Copy link
Member

They seem to have explicitly added the language to two shorthands for some reason, despite that not being common practice.

@SebastianZ
Copy link
Contributor

The actual issue here is that shorthand props don't have initial values, so the 'flex' block is wrong - it should just say "see individual properties", like it does for all the other rows.

I've opened issue #223 to discuss this.

Sebastian

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.

4 participants