Skip to content

Page updates: column-rule-*#44755

Merged
estelle merged 6 commits into
mainfrom
gaps6-column-rule
Jul 21, 2026
Merged

Page updates: column-rule-*#44755
estelle merged 6 commits into
mainfrom
gaps6-column-rule

Conversation

@estelle

@estelle estelle commented Jul 16, 2026

Copy link
Copy Markdown
Member

This PR includes a complete revision of four existing pages:

  • column-rule
  • column-rule-style
  • column-rule-width

The CSS gaps module page is already live.
That previous PR included the task of "Update pages to correct module (gap has been moved from mod to mod)"

This starts the section of openwebdocs/project#238 of "Update pages to new definition"

@estelle
estelle requested a review from a team as a code owner July 16, 2026 19:49
@estelle
estelle requested review from hamishwillee and removed request for a team July 16, 2026 19:49
@github-actions github-actions Bot added Content:CSS Cascading Style Sheets docs size/l [PR only] 501-1000 LoC changed labels Jul 16, 2026
@estelle estelle mentioned this pull request Jul 16, 2026
62 tasks
@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor
Preview URLs (6 pages)
Flaws (24)

Note! 1 document with no flaws that don't need to be listed. 🎉

Found an unexpected or unresolvable flaw? Please report it here.

URL: /en-US/docs/Web/CSS/Reference/Properties/column-rule
Title: column-rule CSS property
Flaw count: 4

  • macros:
    • Macro cssxref produces link /en-US/docs/Web/CSS/line-width which doesn't resolve
    • Macro cssxref produces link /en-US/docs/Web/CSS/rule which doesn't resolve
    • Macro csssyntax produces link /en-US/docs/Web/CSS/Reference/Values/number-token which doesn't resolve
    • Macro cssxref produces link /en-US/docs/Web/CSS/rule which doesn't resolve

URL: /en-US/docs/Web/CSS/Reference/Properties/column-rule-color
Title: column-rule-color CSS property
Flaw count: 4

  • macros:
    • Macro cssxref produces link /en-US/docs/Web/CSS/rule-color which doesn't resolve
    • Macro csssyntax produces link /en-US/docs/Web/CSS/Reference/Values/number-token which doesn't resolve
    • Macro cssxref produces link /en-US/docs/Web/CSS/rule-color which doesn't resolve
    • Macro cssxref produces link /en-US/docs/Web/CSS/rule which doesn't resolve

URL: /en-US/docs/Web/CSS/Reference/Properties/column-rule-style
Title: column-rule-style CSS property
Flaw count: 4

  • macros:
    • Macro cssxref produces link /en-US/docs/Web/CSS/rule-style which doesn't resolve
    • Macro csssyntax produces link /en-US/docs/Web/CSS/Reference/Values/number-token which doesn't resolve
    • Macro cssxref produces link /en-US/docs/Web/CSS/rule-style which doesn't resolve
    • Macro cssxref produces link /en-US/docs/Web/CSS/rule which doesn't resolve

URL: /en-US/docs/Web/CSS/Reference/Properties/column-rule-width
Title: column-rule-width CSS property
Flaw count: 8

  • macros:
    • Macro cssxref produces link /en-US/docs/Web/CSS/line-width which doesn't resolve
    • Macro cssxref produces link /en-US/docs/Web/CSS/line-width which doesn't resolve
    • Macro cssxref produces link /en-US/docs/Web/CSS/rule-width which doesn't resolve
    • Macro cssxref produces link /en-US/docs/Web/CSS/line-width which doesn't resolve
    • Macro cssxref produces link /en-US/docs/Web/CSS/line-width which doesn't resolve
    • and 3 more flaws omitted

URL: /en-US/docs/Web/CSS/Reference/Properties/row-rule-style
Title: row-rule-style CSS property
Flaw count: 4

  • macros:
    • Macro cssxref produces link /en-US/docs/Web/CSS/rule-style which doesn't resolve
    • Macro csssyntax produces link /en-US/docs/Web/CSS/Reference/Values/number-token which doesn't resolve
    • Macro cssxref produces link /en-US/docs/Web/CSS/rule-style which doesn't resolve
    • Macro cssxref produces link /en-US/docs/Web/CSS/rule which doesn't resolve

(comment last updated: 2026-07-21 00:06:01)

Comment thread files/en-us/web/css/reference/properties/column-rule-color/index.md
Comment thread files/en-us/web/css/reference/properties/column-rule-color/index.md
Comment thread files/en-us/web/css/reference/properties/column-rule-color/index.md
Comment thread files/en-us/web/css/reference/properties/column-rule-color/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-style/index.md
Comment thread files/en-us/web/css/reference/properties/column-rule-style/index.md
Comment thread files/en-us/web/css/reference/properties/column-rule-style/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-width/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-width/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-width/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-width/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-style/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-color/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-style/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-color/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-style/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-width/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-color/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-style/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-width/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule/index.md Outdated

@hamishwillee hamishwillee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@estelle Do you have Claude? Running a prompt such as "Review this for spelling and grammar issues" caught quite a few typos.

Still going on my own review ("does it make sense to me as a reader")


We start by defining the list as a grid container, creating columns with the {{cssxref("grid-template-columns")}} property. We include a {{cssxref("gap")}} of `7px` to provide enough room between the columns to fit our `3px` dashed rule, and remove the bullets with {{cssxref("list-style-type")}} set to `none`.

Then, to demonstrate how values can become complicated and the utility of the `repeat()` function, we declare two custom properties, which we use in three {{cssxref("color-mix()")}} color function declarations to create blue, red, and yellow colors. The yellow `color-mix()` color is within a `repeat()` function, set to repeat 3 times.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FWIW I think using the color mix here distracts from the example - its just extra noise.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's actually intentional to show that it helps make ridiculous code less ridiculous

Comment thread files/en-us/web/css/reference/properties/column-rule-style/index.md
Comment thread files/en-us/web/css/reference/properties/column-rule-style/index.md
Comment thread files/en-us/web/css/reference/properties/column-rule-color/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-width/index.md
Comment thread files/en-us/web/css/reference/properties/column-rule-width/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule/index.md
Comment thread files/en-us/web/css/reference/properties/column-rule-style/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-width/index.md Outdated

@hamishwillee hamishwillee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@estelle I copied some but not all fix suggestions from #44725 - please make sure the common problem fixes are captured. Chris and I found similar issues - up to you, but I generally prefer his suggestions.

Comment thread files/en-us/web/css/reference/properties/column-rule-width/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/column-rule-width/index.md Outdated
@estelle
estelle requested a review from hamishwillee July 18, 2026 22:55

- `<repeat-line-style>`
- : A {{cssxref("repeat()")}} function, with the first argument being an {{cssxref("&lt;integer&gt;")}} of `1` or more, and one or more {{cssxref("&lt;line-width&gt;")}} values as subsequent arguments. The integer defines the number of times the `<line-width>` values should be repeated.
- : A {{cssxref("repeat()")}} function, with the first argument being an {{cssxref("&lt;integer&gt;")}} of `1` or more, and subsequent arguments being {{cssxref("&lt;line-style&gt;")}} values. The integer specifies how many times the `<line-style>` values should be repeated.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo!

@hamishwillee hamishwillee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@estelle Thanks. I ran Claude again for consistency against Chris' suggestions in the other PR and submitted as d047891 - they all made sense to me.

On scan I didn't see any other changes worth making

Approving but not merging so you can sanity check the last commit if you want.

@estelle
estelle merged commit 5cf8432 into main Jul 21, 2026
13 checks passed
@estelle
estelle deleted the gaps6-column-rule branch July 21, 2026 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:CSS Cascading Style Sheets docs size/l [PR only] 501-1000 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants