Skip to content

[css-flexbox-1] Typo: s/specified size/specified size suggestion/#5349

Merged
svgeesus merged 1 commit into
masterfrom
Loirooriol-patch-1
Sep 6, 2020
Merged

[css-flexbox-1] Typo: s/specified size/specified size suggestion/#5349
svgeesus merged 1 commit into
masterfrom
Loirooriol-patch-1

Conversation

@Loirooriol

Copy link
Copy Markdown
Contributor

However, if the box has an aspect ratio and no <a>specified size</a>,

The spec refers to https://drafts.csswg.org/css-flexbox/#specified-size-suggestion
But currently it's linking to https://drafts.csswg.org/css-images-3/#specified-size

@Loirooriol Loirooriol added the css-flexbox-1 Current Work label Jul 22, 2020
@Loirooriol Loirooriol requested review from fantasai and tabatkins July 22, 2020 14:32
@svgeesus svgeesus self-requested a review September 6, 2020 09:19
Comment thread css-flexbox-1/Overview.bs

In general, the <a>content-based minimum size</a> of a <a>flex item</a>
is the smaller of its <a>content size suggestion</a> and its <a>specified size suggestion</a>.
However, if the box has an aspect ratio and no <a>specified size</a>,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes, clearly

@svgeesus svgeesus merged commit 561d918 into master Sep 6, 2020
@svgeesus svgeesus deleted the Loirooriol-patch-1 branch September 6, 2020 09:20
@syncbot syncbot restored the Loirooriol-patch-1 branch October 23, 2020 07:11
@plinss plinss deleted the Loirooriol-patch-1 branch October 28, 2020 16:57
@syncbot syncbot restored the Loirooriol-patch-1 branch October 28, 2020 17:29
@plinss plinss deleted the Loirooriol-patch-1 branch October 28, 2020 17:30
@syncbot syncbot restored the Loirooriol-patch-1 branch October 28, 2020 18:02
@plinss plinss deleted the Loirooriol-patch-1 branch October 28, 2020 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

css-flexbox-1 Current Work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants