Skip to content

Update deprecated margin attributes on body#43534

Merged
sideshowbarker merged 1 commit intomdn:mainfrom
zcorpan:patch-7
Mar 27, 2026
Merged

Update deprecated margin attributes on body#43534
sideshowbarker merged 1 commit intomdn:mainfrom
zcorpan:patch-7

Conversation

@zcorpan
Copy link
Copy Markdown
Contributor

@zcorpan zcorpan commented Mar 23, 2026

Description

Update the descriptions for these attributes to match the spec change.

Motivation

MDN should match the spec.

Additional details

See whatwg/html#11881

Related issues and pull requests

@zcorpan zcorpan requested a review from a team as a code owner March 23, 2026 14:53
@github-actions github-actions bot added Content:HTML Hypertext Markup Language docs size/s [PR only] 6-50 LoC changed labels Mar 23, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Preview URLs (1 page)

@pepelsbey
Copy link
Copy Markdown
Member

MDN should match the spec.

Not exactly 😬 We document browser behavior for real-world usage, that’s only based on specs. But the implementation might be different.

So I would test the implementation and only then update the docs.

@sideshowbarker sideshowbarker merged commit 2a7d7d2 into mdn:main Mar 27, 2026
7 checks passed
@pepelsbey
Copy link
Copy Markdown
Member

@sideshowbarker have you seen my comment?

@sideshowbarker
Copy link
Copy Markdown
Member

@sideshowbarker have you seen my comment?

Sorry no, I somehow missed it

@zcorpan
Copy link
Copy Markdown
Contributor Author

zcorpan commented Mar 27, 2026

@pepelsbey fair, but the recommendation for these attributes is "don't use". Is it helpful to explain how these attributes were previously non-interoperable?

@pepelsbey
Copy link
Copy Markdown
Member

Is it helpful to explain how these attributes were previously non-interoperable?

@zcorpan I'm only saying that if they are supported in browsers for backward compatibility reasons, we should reflect that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:HTML Hypertext Markup Language docs size/s [PR only] 6-50 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants