Skip to content

updating min-content and max-content support information for Firefox … - #3573

Closed
chrisdavidmills wants to merge 1 commit into
mdn:masterfrom
chrisdavidmills:min-content-max-content-unprefix
Closed

updating min-content and max-content support information for Firefox …#3573
chrisdavidmills wants to merge 1 commit into
mdn:masterfrom
chrisdavidmills:min-content-max-content-unprefix

Conversation

@chrisdavidmills

Copy link
Copy Markdown
Contributor

…now it is available unprefixed

Firefox now supports the max-content and min-content keywords unprefixed for all of the properties listed at https://bugzilla.mozilla.org/show_bug.cgi?id=1322780#c60.

This is an attempt to update them. I also did a bit of breaking out these keywords (and fit-content) onto separate support data lines where they were represeted all in one item because it just didn't make sense to have them lumped together.

@Elchi3 Elchi3 added the data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS label Mar 6, 2019
@ddbeck

ddbeck commented Mar 6, 2019

Copy link
Copy Markdown
Contributor

Just a heads up here, but I think there's going to be a conflict with #3457

@chrisdavidmills

Copy link
Copy Markdown
Contributor Author

@ddbeck ack, dammnit, I forgot Rachel did this!

Well, I'd much rather merge Rachel's PR first, then perhaos close mine and just add in any missing details I uncovered. It looks like she did mostly the same thing as me, but with some better testing across browsers.

@ddbeck

ddbeck commented Mar 6, 2019

Copy link
Copy Markdown
Contributor

Yeah, I think the combination of your two PRs will be great (in particular, I think you picked up some prefix data that's not in Rachel's PR). I'll mark this as not-ready for now and you can decide whether to close or risk a rebase after Rachel's PR is merged

@ddbeck ddbeck added the not ready This is not yet ready to be merged. It's pending a decision, other PR, or a prerequisite action. label Mar 6, 2019
@ddbeck

ddbeck commented Mar 7, 2019

Copy link
Copy Markdown
Contributor

#3457 has been merged, @chrisdavidmills. I'll leave it to you whether to rebase or open a new PR. Thank you!

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

Labels

data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS not ready This is not yet ready to be merged. It's pending a decision, other PR, or a prerequisite action.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants