Skip to content

Commit 52582d3

Browse files
committed
rustdoc: remove empty extern_crates and type="text/javascript" on script
Like #101023, this removes an attribute with a default value.
1 parent 42fa8ac commit 52582d3

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

src/librustdoc/html/render/print_item.rs

+4-2
Original file line numberDiff line numberDiff line change
@@ -987,7 +987,7 @@ fn item_trait(w: &mut Buffer, cx: &mut Context<'_>, it: &clean::Item, t: &clean:
987987
// So C's HTML will have something like this:
988988
//
989989
// ```html
990-
// <script type="text/javascript" src="/implementors/A/trait.Foo.js"
990+
// <script src="/implementors/A/trait.Foo.js"
991991
// data-ignore-extern-crates="A,B" async></script>
992992
// ```
993993
//
@@ -1013,9 +1013,11 @@ fn item_trait(w: &mut Buffer, cx: &mut Context<'_>, it: &clean::Item, t: &clean:
10131013
.map(|cnum| cx.shared.tcx.crate_name(cnum).to_string())
10141014
.collect::<Vec<_>>()
10151015
.join(",");
1016+
let (extern_before, extern_after) =
1017+
if extern_crates.is_empty() { ("", "") } else { (" data-ignore-extern-crates=\"", "\"") };
10161018
write!(
10171019
w,
1018-
"<script type=\"text/javascript\" src=\"{src}\" data-ignore-extern-crates=\"{extern_crates}\" async></script>",
1020+
"<script src=\"{src}\"{extern_before}{extern_crates}{extern_after} async></script>",
10191021
src = js_src_path.finish(),
10201022
);
10211023
}

src/librustdoc/html/static/js/main.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -528,9 +528,9 @@ function loadCss(cssFileName) {
528528
// We don't want to include impls from this JS file, when the HTML already has them.
529529
// The current crate should always be ignored. Other crates that should also be
530530
// ignored are included in the attribute `data-ignore-extern-crates`.
531-
const ignoreExternCrates = document
532-
.querySelector("script[data-ignore-extern-crates]")
533-
.getAttribute("data-ignore-extern-crates");
531+
const script = document
532+
.querySelector("script[data-ignore-extern-crates]");
533+
const ignoreExternCrates = script ? script.getAttribute("data-ignore-extern-crates") : "";
534534
for (const lib of libs) {
535535
if (lib === window.currentCrate || ignoreExternCrates.indexOf(lib) !== -1) {
536536
continue;

0 commit comments

Comments
 (0)