Editorial review: Document CSS multicol column wrapping#42934
Editorial review: Document CSS multicol column wrapping#42934chrisdavidmills merged 55 commits intomdn:mainfrom
Conversation
Preview URLs (7 pages)
Flaws (13)Note! 3 documents with no flaws that don't need to be listed. 🎉 Found an unexpected or unresolvable flaw? Please report it here. URL:
URL:
URL:
URL:
(comment last updated: 2026-03-04 20:42:49) |
rachelandrew
left a comment
There was a problem hiding this comment.
This looks generally ok, other than a nit on the overflow page and a suggestion to make things clearer around what causes columns to overflow.
files/en-us/web/css/guides/multicol_layout/handling_overflow/index.md
Outdated
Show resolved
Hide resolved
files/en-us/web/css/guides/multicol_layout/handling_overflow/index.md
Outdated
Show resolved
Hide resolved
files/en-us/web/css/reference/properties/column-height/index.md
Outdated
Show resolved
Hide resolved
…ndex.md Co-authored-by: Rachel Andrew <me@rachelandrew.co.uk>
estelle
left a comment
There was a problem hiding this comment.
lots of comments, but nothing major
files/en-us/web/css/guides/multicol_layout/handling_overflow/index.md
Outdated
Show resolved
Hide resolved
files/en-us/web/css/guides/multicol_layout/handling_overflow/index.md
Outdated
Show resolved
Hide resolved
files/en-us/web/css/guides/multicol_layout/handling_overflow/index.md
Outdated
Show resolved
Hide resolved
files/en-us/web/css/guides/multicol_layout/handling_overflow/index.md
Outdated
Show resolved
Hide resolved
files/en-us/web/css/guides/multicol_layout/handling_overflow/index.md
Outdated
Show resolved
Hide resolved
…ndex.md Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
…ndex.md Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
…ndex.md Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
…ndex.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
estelle
left a comment
There was a problem hiding this comment.
-I think height: fit-content can be removed from the no-support sections.
- Recommend placing relevant CSS before hidden CSS in the source order so hitting "play" shows the relevant to the example CSS at the top. If there are specificity concerns, put the hidden CSS in a layer.
- Begin the description section with the name of the feature and a summary description.
- A lot of the suggestions are to make the reference pages sound less guide-like and more reference-esque.
files/en-us/web/css/reference/properties/column-height/index.md
Outdated
Show resolved
Hide resolved
files/en-us/web/css/reference/properties/column-height/index.md
Outdated
Show resolved
Hide resolved
files/en-us/web/css/reference/properties/column-height/index.md
Outdated
Show resolved
Hide resolved
files/en-us/web/css/reference/properties/column-height/index.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
estelle
left a comment
There was a problem hiding this comment.
Looks good. Just a few minor tweaks, but I don't need to re-review. approving.
files/en-us/web/css/reference/properties/column-height/index.md
Outdated
Show resolved
Hide resolved
files/en-us/web/css/reference/properties/column-height/index.md
Outdated
Show resolved
Hide resolved
files/en-us/web/css/reference/properties/column-height/index.md
Outdated
Show resolved
Hide resolved
|
|
||
| #### CSS | ||
|
|
||
| We start by setting {{cssxref("column-width")}} on the {{htmlelement("body")}} element to define the preferred width for the columns. A {{cssxref("gap")}} of `3em 2em` results in a `3em` gap between rows and a `2em` gap between columns. The {{cssxref("column-rule")}} adds a line in the center of the gap between the columns. The `column-height` of `95vh` makes the columns nearly as tall as the viewport. |
There was a problem hiding this comment.
I did a PR if you want to do a quick review - #43338
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
These comments were handled.
Description
Chrome 145 adds support for Column wrapping for multicol, which includes the
column-heightandcolumn-wrapproperties. See https://chromestatus.com/feature/5176206485618688.This PR documents those two properties and adds appropriate mentions in other places.
Motivation
Additional details
Related issues and pull requests
Fixes #43183