Skip to content

Commit

Permalink
Rollup merge of rust-lang#103890 - notriddle:notriddle/mobile-rustdoc…
Browse files Browse the repository at this point in the history
…-padding-top, r=GuillaumeGomez

rustdoc: remove unused mobile CSS `.rustdoc { padding-top: 0 }`

When this rule was added in dd437ee, as `body { padding-top: 0 }`, the desktop body tag had non-zero top padding. This padding was removed in 135281e.

This rule no longer overrides a rule in rustdoc's desktop styles, and also doesn't override the UA stylesheet, since the [HTML standard] has only margin, not padding, on the page body.

[HTML standard]: https://html.spec.whatwg.org/multipage/rendering.html#the-page
  • Loading branch information
matthiaskrgr authored Nov 2, 2022
2 parents 88f9f49 + deb6538 commit 36d8134
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion src/librustdoc/html/static/css/rustdoc.css
Original file line number Diff line number Diff line change
Expand Up @@ -1677,7 +1677,6 @@ in storage.js
}

.rustdoc {
padding-top: 0px;
/* Sidebar should overlay main content, rather than pushing main content to the right.
Turn off `display: flex` on the body element. */
display: block;
Expand Down

0 comments on commit 36d8134

Please sign in to comment.