Skip to content

[CSSGapDecorations] Refresh explainer with new outset properties#1180

Merged
kbabbitt merged 6 commits intoMicrosoftEdge:mainfrom
jav099:main
Feb 18, 2026
Merged

[CSSGapDecorations] Refresh explainer with new outset properties#1180
kbabbitt merged 6 commits intoMicrosoftEdge:mainfrom
jav099:main

Conversation

@jav099
Copy link
Copy Markdown
Contributor

@jav099 jav099 commented Oct 27, 2025

@jav099
Copy link
Copy Markdown
Contributor Author

jav099 commented Nov 10, 2025

@microsoft-github-policy-service agree company="Microsoft"

Copy link
Copy Markdown
Member

@kbabbitt kbabbitt left a comment

Choose a reason for hiding this comment

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

I was looking this morning at updating the explainer and came across this PR which completely fell off my radar, sorry about that. @jav099 could you switch "outset" to "inset" throughout and then we can land it?

Comment thread CSSGapDecorations/explainer.md Outdated
Comment on lines 219 to 220
@@ -220,25 +220,52 @@ The "zero point" is the edge of the intersection, with positive values extending
into the intersection and negative values receding from it.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
into the intersection and negative values receding from it.
The "zero point" is the edge of the intersection, with negative values extending
into the intersection and positive values receding from it.

Comment thread CSSGapDecorations/explainer.md Outdated
.outset-5px {
column-rule-outset: 5px;
.inset-5px {
column-rule-inset: 5px;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we need to flip signs on inset values in example text? And/or update example image names to reflect the insets they're depicting?

@kbabbitt kbabbitt merged commit 8c97182 into MicrosoftEdge:main Feb 18, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants