Skip to content

New pages: rule-* shorthand properties - #44895

Open
estelle wants to merge 7 commits into
mainfrom
gaps6-shorthandrules
Open

New pages: rule-* shorthand properties#44895
estelle wants to merge 7 commits into
mainfrom
gaps6-shorthandrules

Conversation

@estelle

@estelle estelle commented Jul 23, 2026

Copy link
Copy Markdown
Member

This PR includes a the creation of three new reference pages:

  • rule-style
  • rule-color
  • rule-width

The CSS gaps module page is already live.

Part of openwebdocs/project#238

@estelle
estelle requested a review from a team as a code owner July 23, 2026 11:22
@estelle
estelle requested review from chrisdavidmills and removed request for a team July 23, 2026 11:22
@github-actions github-actions Bot added Content:CSS Cascading Style Sheets docs size/xl [PR only] >1000 LoC changed labels Jul 23, 2026
@estelle estelle mentioned this pull request Jul 23, 2026
65 tasks
@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Preview URLs (3 pages)

Flaws (10)

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

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

  • macros:
    • 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/rule-style
Title: rule-style CSS property
Flaw count: 4

  • macros:
    • 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
    • Macro cssxref produces link /en-US/docs/Web/CSS/rule which doesn't resolve

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

  • macros:
    • 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

(comment last updated: 2026-07-28 11:26:46)

@chrisdavidmills chrisdavidmills left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good work, @estelle. A bunch of grammar corrections and such, but nothing major.

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

#### HTML

We create a list of 100 items. Most of the HTML is hidden for brevity.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What was your reasoning behind using this example, rather than the 75-item list used in the other two pages? It isn't a massive problem, but this one seems uglier and clumsier than the other one.

It is also taller than the 75-item example, meaning that the live examples all have scrollbars.

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.

i did this one first, then thought 75 was better. Will change this one.

Comment thread files/en-us/web/css/reference/properties/rule-width/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/rule-width/index.md Outdated
Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
@estelle
estelle requested a review from a team as a code owner July 28, 2026 11:10
@estelle
estelle requested review from dipikabh and removed request for a team July 28, 2026 11:10
Updated examples and descriptions for clarity regarding the grid container dimensions and the use of the repeat() function in the rule-width property.

@estelle estelle left a comment

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.

i think i got everything


{{EmbedLiveSample("Basic", "", "600")}}

### Multiple values

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.

no. i happen to have excess values, but the point is to show more than one value.


#### HTML

We create a list of 100 items. Most of the HTML is hidden for brevity.

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.

i did this one first, then thought 75 was better. Will change this one.

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

This example demonstrates using the `repeat()` function within the `rule-style` property value. We use the same HTML and CSS as in the previous examples. We include a `repeat()` function, setting the list of two `<line-style>` values to be repeated 3 times.

```css live-sample___func live-sample___auto

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.

good catch. originally there was more content, and i didn't remove the second example when i removed the features

Comment thread files/en-us/web/css/reference/properties/rule-width/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/rule-width/index.md Outdated
Comment thread files/en-us/web/css/reference/properties/rule-color/index.md Outdated
@estelle
estelle requested a review from chrisdavidmills July 28, 2026 11:25

@chrisdavidmills chrisdavidmills left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@estelle, a few more bits for you. I think you can get it merged after fixing these, so I'll approve. Great work!


## Examples

### Basic example

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

FYI, I had a look in Fx, and this example isn't wired up to the non-support banner.


#### Result

{{EmbedLiveSample("Multiple", "", "600")}}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This example DOES show the non-support banner in Firefox, but it isn't absolutely positioned, meaning that it sits at the top of the other content and causes a scrollbar. It would look tidier if it was abspos'ed, like it is on some of the banners we've been using in other places.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same comment for the other two examples below.


#### Result

{{EmbedLiveSample("Basic", "", "600")}}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This example isn't wired up to the non-support banner either. Tested in Fx.


#### Result

{{EmbedLiveSample("Multiple", "", "600")}}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Again, it would be good for the non-support banners to be abspos'ed.


#### HTML

We create a list of 100 items. Most of the HTML is hidden for brevity.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
We create a list of 100 items. Most of the HTML is hidden for brevity.
We create a list of 75 items. Most of the HTML is hidden for brevity.


#### Result

{{EmbedLiveSample("Basic", "", "600")}}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same comment about abspos for the non-support banners.


{{EmbedLiveSample("func", "", "600")}}

The `repeat()` function repeats two width values four times, creating a list of ten width values. As there are fewer column and row gutters than total widths, the last values in the list is discarded.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
The `repeat()` function repeats two width values four times, creating a list of ten width values. As there are fewer column and row gutters than total widths, the last values in the list is discarded.
The `repeat()` function repeats two width values four times, creating a list of ten width values. As there are fewer column and row gutters than total widths, the last values in the list are discarded.

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/xl [PR only] >1000 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants