diff options
| author | Jacob Hoffman-Andrews <github@hoffman-andrews.com> | 2022-02-01 22:11:36 -0800 |
|---|---|---|
| committer | Jacob Hoffman-Andrews <github@hoffman-andrews.com> | 2022-02-02 00:29:33 -0800 |
| commit | 32f62607c3142dfc9eb56a0bd72dee298ca43358 (patch) | |
| tree | ddba653b35dcf51d20dbe9c0df312b05e0ed38a2 /src/test/rustdoc/version-separator-without-source.rs | |
| parent | 1ea4851715893ee3f365a8ef09d47165e9a7864f (diff) | |
| download | rust-32f62607c3142dfc9eb56a0bd72dee298ca43358.tar.gz rust-32f62607c3142dfc9eb56a0bd72dee298ca43358.zip | |
Emit valid HTML from rustdoc
Previously, tidy-html5 (`tidy`) would complain about a few things in our HTML. The main thing is that `<summary>` tags can't contain `<div>`s. That's easily fixed by changing out the `<div>`s for `<span>`s with `display: block`. However, there's also a rule that `<span>`s can't contain heading elements. `<span>` permits only "phrasing content" https://developer.mozilla.org/en-US/docs/Web/HTML/Element/span, and `<h3>` (and friends) are "Flow content, heading content, palpable content". https://developer.mozilla.org/en-US/docs/Web/HTML/Element/Heading_Elements We have a wrapping `<div>` that goes around each `<h3>`/`<h4>`, etc. We turn that into a `<section>` rather than a `<span>` because `<section>` permits "flow content". https://developer.mozilla.org/en-US/docs/Web/HTML/Element/section After this change we get only three warnings from tidy, run on struct.String.html: line 6 column 10790 - Warning: trimming empty <span> line 1 column 1118 - Warning: <link> proprietary attribute "disabled" line 1 column 1193 - Warning: <link> proprietary attribute "disabled" The empty `<span>` is a known issue - there's a span in front of the search box to work around a strange Safari issue. The `<link>` attributes are the non-default stylesheets. We can probably refactor theme application to avoid using this proprietary "disabled" attribute.
Diffstat (limited to 'src/test/rustdoc/version-separator-without-source.rs')
| -rw-r--r-- | src/test/rustdoc/version-separator-without-source.rs | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/test/rustdoc/version-separator-without-source.rs b/src/test/rustdoc/version-separator-without-source.rs index bffe5030a84..ae866deba1e 100644 --- a/src/test/rustdoc/version-separator-without-source.rs +++ b/src/test/rustdoc/version-separator-without-source.rs @@ -16,8 +16,8 @@ pub fn foo() {} pub struct Bar; impl Bar { - // @has - '//div[@id="method.bar"]/*[@class="rightside"]' '2.0' - // @!has - '//div[@id="method.bar"]/*[@class="rightside"]' '2.0 ·' + // @has - '//*[@id="method.bar"]/*[@class="rightside"]' '2.0' + // @!has - '//*[@id="method.bar"]/*[@class="rightside"]' '2.0 ·' #[stable(feature = "foobar", since = "2.0")] pub fn bar() {} } |
