Bug 239450

Summary:

[contain-intrinsic-size] Support contain-intrinsic-size auto && <length>

Product:

WebKit

Reporter:

cathiechen <cathiechen>

Component:

CSS

Assignee:

cathiechen <cathiechen>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Local Build

Hardware:

Unspecified

OS:

Unspecified

See Also:

https://github.com/w3c/csswg-drafts/issues/6308

Bug Depends on:

248998

Bug Blocks:

236707, 249919

Attachments:

Description	Flags
WIP-patch	none
WIP-patch	none
WIP-patch	none
WIP-patch	none
WIP-patch	none

Description cathiechen 2022-04-17 21:10:13 PDT

Per [1], the last remembers size of contain-intrinsic-size auto && <length> is only effective when content-visibility: auto

[1] https://github.com/w3c/csswg-drafts/issues/6308

Comment 1 cathiechen 2022-04-18 00:27:43 PDT

Created attachment 457796 [details]
WIP-patch

Comment 2 cathiechen 2022-04-18 00:54:09 PDT

Created attachment 457798 [details]
WIP-patch

Comment 3 Radar WebKit Bug Importer 2022-04-24 21:11:13 PDT

<rdar://problem/92246792>

Comment 4 cathiechen 2022-04-29 03:05:04 PDT

Created attachment 458578 [details]
WIP-patch

Comment 5 cathiechen 2022-10-11 07:37:01 PDT

(In reply to cathiechen from comment #0)
> Per [1], the last remembers size of contain-intrinsic-size auto && <length>
> is only effective when content-visibility: auto
> 
> [1] https://github.com/w3c/csswg-drafts/issues/6308

Update the conclusion:

> Drop the "not relevant to the user" condition

Now, the condition is "currently skipping its contents", so it would work for both content-visibility: auto and content-visibility: hidden.

Comment 6 cathiechen 2022-10-31 12:13:58 PDT

Created attachment 463327 [details]
WIP-patch

Comment 7 cathiechen 2022-11-23 10:48:48 PST

Created attachment 463694 [details]
WIP-patch

Comment 8 cathiechen 2022-12-18 12:04:42 PST

Pull request: https://github.com/WebKit/WebKit/pull/7829

Comment 9 EWS 2023-02-20 04:48:21 PST

Committed 260535@main (88587e7bd545): <https://commits.webkit.org/260535@main>

Reviewed commits have been landed. Closing PR #7829 and removing active labels.