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.

Sign up for GitHub

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

Jump to bottom

[css-sizing-3] max-content contribution of replaced only-aspect-ratio elements might be improvable #4218

Closed
tabatkins opened this issue Aug 19, 2019 · 3 comments
Closed

[css-sizing-3] max-content contribution of replaced only-aspect-ratio elements might be improvable #4218

tabatkins opened this issue Aug 19, 2019 · 3 comments
Labels
Closed Accepted by CSSWG Resolution css-sizing-3 Current Work Needs Testcase (WPT)

Comments

@tabatkins
Copy link
Member

We appear to have no interop on this case:

<!DOCTYPE html>
<div style="width: max-content; border: thick double;">
<svg style="float:left;" viewBox="0 0 200 100"></svg>

Chrome gives it 0x0, Firefox gives it ICB width and then ratio height. The spec says 300px width and ratio height.

We suggest to go with FF's behavior, as this seems more useful than 0, and less arbitrary than 300px.
@tabatkins tabatkins added the css-sizing-3 Current Work label Aug 19, 2019
@fantasai fantasai mentioned this issue Aug 20, 2019
Open
@jensimmons
Copy link
Contributor

I agree.

@css-meeting-bot
Copy link
Member

The CSS Working Group just discussed [css-sizing-3] max-content contribution of replaced only-ar elements might be improvable, and agreed to the following:

  • RESOLVED: Specify Firefox's behavior
The full IRC log of that discussion <dael> Topic: [css-sizing-3] max-content contribution of replaced only-ar elements might be improvable
<dael> github: https://github.com//issues/4218
<dael> fantasai: We triaged this last week. Case is we have some box which says I want width max-content
<dael> fantasai: Something inside it intrinsically sized with an aspect ratio and no size info
<dael> fantasai: No interop. Chrome is 0x0 as max content. Spec says use 300px width and ratio for height. Resolvable but arbitratry. FF uses initial containing block. That seems useful. We suggest spec FF behavior
<dael> astearns: Concerns?
<dael> astearns: Objections to Specify Firefox's behavior?
<dael> RESOLVED: Specify Firefox's behavior

@smfr smfr changed the title [css-sizing-3] max-content contribution of replaced only-ar elements might be improvable [css-sizing-3] max-content contribution of replaced only-aspect-ratio elements might be improvable Jan 15, 2020
@mozilla-apprentice mozilla-apprentice mentioned this issue Jan 15, 2020
Open
@fantasai
Copy link
Collaborator

Fixed in 57e2680

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Closed Accepted by CSSWG Resolution css-sizing-3 Current Work Needs Testcase (WPT)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@jensimmons @tabatkins @fantasai @css-meeting-bot