Closed
Bug 1463594
(css-contain-size)
Opened 6 years ago
Closed 5 years ago
[meta] Implement CSS "contain: size"
Categories
(Core :: Layout, enhancement, P3)
Core
Layout
Tracking
()
RESOLVED
FIXED
People
(Reporter: dholbert, Unassigned)
References
(Depends on 3 open bugs, )
Details
(Keywords: dev-doc-complete, meta, Whiteboard: [layout:p1])
See https://drafts.csswg.org/css-contain/#containment-size This bug is on making our layout/reflow code react appropriately to "contain: size". Historical note: the behavior for "contain:size" was lumped in with layout containment in an earlier version of the spec, so I think part of this was included in the work-in-progress patch stack on bug 1178895. But it's now been split out into its own keyword, which can be specified independently from (or together with) "contain:layout". (Note that bug 1463589 will add CSS Parser support for "contain:size".)
Reporter | ||
Updated•6 years ago
|
Assignee: nobody → mreschenberg
Blocks: css-contain-1
Reporter | ||
Updated•6 years ago
|
status-firefox62:
affected → ---
Comment 1•6 years ago
|
||
5 'inline-block' with 'contain: size' tests: http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/contain-size-006.xht http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/contain-size-007.xht http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/contain-size-008.xht http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/contain-size-009.xht http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/contain-size-010.xht More tests will come. Right now, Chromium 66.0.3359.117 and Chrome 68.0.3438.3 pass those 5 tests. Firefox 62.0a1 buildID=20180613220131 with layout.css.contain.enabled set to true fails those 5 tests.
Reporter | ||
Comment 2•6 years ago
|
||
Thanks, Gérard!
Comment 3•6 years ago
|
||
Visit http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/ for inline-block tests (8 tests: from contain-size-006 to 013) and for inline-table tests (expecting 8 tests: from contain-size-021 to 028). Reference files: http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/contain-size-006-ref.xht http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/contain-size-007-ref.xht
Comment 4•6 years ago
|
||
> http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/ > > for inline-block tests (8 tests: from contain-size-006 to 013) and > > for inline-table tests (expecting 8 tests: from contain-size-021 to 028). Those tests' filename numbers will unfortunately have to be changed because tests in https://bugs.chromium.org/p/chromium/issues/detail?id=843329#c6 are already using those filename numbers.
Comment 5•6 years ago
|
||
Tests with *new* filename numbers - - - - - - - - - - - - - - - - - http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/ for inline-block tests (8 tests: from contain-size-021 to 028) New reference files: http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/contain-size-021-ref.xht http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/contain-size-022-ref.xht for inline-table tests (expecting 8 tests: from contain-size-031 to 038) (I must reconsider those tests based on the fact that 'contain: size' does not apply to table elements and presumably redo these tests.) for inline replaced element tests: contain-size-041 and contain-size-042 for table-cell tests: contain-size-051 and contain-size-052 for block box tests: 1 test for now: contain-size-061
Depends on: 1478550
Comment 6•6 years ago
|
||
(In reply to Gérard Talbot from comment #5) > Tests with *new* filename numbers > - - - - - - - - - - - - - - - - - > > > http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/ > > > > for inline-block tests (8 tests: from contain-size-021 to 028) > > New reference files: > > http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/contain-size-021-ref. > xht > > http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/contain-size-022-ref. > xht Now at: http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/contain-size-021-ref.html http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/contain-size-022-ref.html I must keep only 1 copy of tests and reference files on my site. I'm dropping XHTML and switching to HTML5 for tests. The links given in 1st, 3rd and 5th comments of this bug report will be broken.
Comment 7•6 years ago
|
||
> http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/contain-size-021-ref. > html > > http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/contain-size-022-ref. > html Now at: http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/reference/contain-size-021-ref.html http://www.gtalbot.org/BrowserBugsSection/CSS3Contain/reference/contain-size-022-ref.html
Updated•6 years ago
|
Updated•6 years ago
|
Whiteboard: [layout:p1]
Updated•6 years ago
|
Keywords: dev-doc-needed
Depends on: 1497718
Updated•5 years ago
|
Assignee: mreschenberg → nobody
Status: ASSIGNED → NEW
Reporter | ||
Comment 8•5 years ago
|
||
This feature shipped in Firefox 69 (with the implementation work happening in dependent bugs), so let's call this [meta] and close it out.
(yay!)
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Summary: Implement CSS "contain: size" → [meta] Implement CSS "contain: size"
Updated•5 years ago
|
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•