Skip to content
This repository was archived by the owner on May 28, 2025. It is now read-only.

Commit caa1d47

Browse files
author
Yuki Okushi
authored
Rollup merge of rust-lang#106819 - notriddle:notriddle/rm-h1-fqn, r=GuillaumeGomez
rustdoc: remove unnecessary DOM class `h1.fqn` It's misleading. The main heading sometimes isn't an fully qualified name at all. It's also redundant. It's always a child of `div.main-heading`, so just use that.
2 parents 7d02116 + 39b90a5 commit caa1d47

17 files changed

+33
-33
lines changed

‎src/librustdoc/html/render/context.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -639,7 +639,7 @@ impl<'tcx> FormatRenderer<'tcx> for Context<'tcx> {
639639
write!(
640640
buf,
641641
"<div class=\"main-heading\">\
642-
<h1 class=\"fqn\">Rustdoc settings</h1>\
642+
<h1>Rustdoc settings</h1>\
643643
<span class=\"out-of-band\">\
644644
<a id=\"back\" href=\"javascript:void(0)\" onclick=\"history.back();\">\
645645
Back\
@@ -677,7 +677,7 @@ impl<'tcx> FormatRenderer<'tcx> for Context<'tcx> {
677677
write!(
678678
buf,
679679
"<div class=\"main-heading\">\
680-
<h1 class=\"fqn\">Rustdoc help</h1>\
680+
<h1>Rustdoc help</h1>\
681681
<span class=\"out-of-band\">\
682682
<a id=\"back\" href=\"javascript:void(0)\" onclick=\"history.back();\">\
683683
Back\

‎src/librustdoc/html/render/mod.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -364,7 +364,7 @@ impl AllTypes {
364364
}
365365
}
366366

367-
f.write_str("<h1 class=\"fqn\">List of all items</h1>");
367+
f.write_str("<h1>List of all items</h1>");
368368
// Note: print_entries does not escape the title, because we know the current set of titles
369369
// doesn't require escaping.
370370
print_entries(f, &self.structs, ItemSection::Structs);
@@ -394,7 +394,7 @@ fn scrape_examples_help(shared: &SharedContext<'_>) -> String {
394394
let mut ids = IdMap::default();
395395
format!(
396396
"<div class=\"main-heading\">\
397-
<h1 class=\"fqn\">About scraped examples</h1>\
397+
<h1>About scraped examples</h1>\
398398
</div>\
399399
<div>{}</div>",
400400
Markdown {

‎src/librustdoc/html/render/write_shared.rs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -345,7 +345,7 @@ if (typeof exports !== 'undefined') {exports.searchIndex = searchIndex};
345345
};
346346

347347
let content = format!(
348-
"<h1 class=\"fqn\">List of all crates</h1><ul class=\"all-items\">{}</ul>",
348+
"<h1>List of all crates</h1><ul class=\"all-items\">{}</ul>",
349349
krates
350350
.iter()
351351
.map(|s| {

‎src/librustdoc/html/static/css/rustdoc.css‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,7 @@ h1, h2, h3, h4 {
139139
.docblock > h6:first-child {
140140
margin-top: 0;
141141
}
142-
h1.fqn {
142+
.main-heading h1 {
143143
margin: 0;
144144
padding: 0;
145145
flex-grow: 1;

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -526,7 +526,7 @@ function loadCss(cssUrl) {
526526
}
527527

528528
let currentNbImpls = implementors.getElementsByClassName("impl").length;
529-
const traitName = document.querySelector("h1.fqn > .trait").textContent;
529+
const traitName = document.querySelector(".main-heading h1 > .trait").textContent;
530530
const baseIdName = "impl-" + traitName + "-";
531531
const libs = Object.getOwnPropertyNames(imp);
532532
// We don't want to include impls from this JS file, when the HTML already has them.

‎src/librustdoc/html/templates/print_item.html‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
<div class="main-heading"> {#- -#}
2-
<h1 class="fqn"> {#- -#}
2+
<h1> {#- -#}
33
{{-typ-}}
44
{#- The breadcrumbs of the item path, like std::string -#}
55
{%- for component in path_components -%}

‎tests/rustdoc-gui/anchors.goml‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
define-function: (
44
"check-colors",
5-
(theme, main_color, title_color, fqn_color, fqn_type_color, src_link_color, sidebar_link_color),
5+
(theme, main_color, title_color, main_heading_color, main_heading_type_color, src_link_color, sidebar_link_color),
66
block {
77
goto: "file://" + |DOC_PATH| + "/staged_api/struct.Foo.html"
88
// This is needed to ensure that the text color is computed.
@@ -14,8 +14,8 @@ define-function: (
1414
reload:
1515

1616
assert-css: ("#toggle-all-docs", {"color": |main_color|})
17-
assert-css: (".fqn a:nth-of-type(1)", {"color": |fqn_color|})
18-
assert-css: (".fqn a:nth-of-type(2)", {"color": |fqn_type_color|})
17+
assert-css: (".main-heading h1 a:nth-of-type(1)", {"color": |main_heading_color|})
18+
assert-css: (".main-heading a:nth-of-type(2)", {"color": |main_heading_type_color|})
1919
assert-css: (
2020
".rightside .srclink",
2121
{"color": |src_link_color|, "text-decoration": "none solid " + |src_link_color|},
@@ -57,7 +57,7 @@ define-function: (
5757
assert-css: ("#top-doc-prose-title", {"color": |title_color|})
5858

5959
assert-css: (".sidebar a", {"color": |sidebar_link_color|})
60-
assert-css: ("h1.fqn a", {"color": |title_color|})
60+
assert-css: (".main-heading h1 a", {"color": |title_color|})
6161

6262
// We move the cursor over the "Implementations" title so the anchor is displayed.
6363
move-cursor-to: "h2#implementations"
@@ -77,8 +77,8 @@ call-function: (
7777
"theme": "ayu",
7878
"main_color": "rgb(197, 197, 197)",
7979
"title_color": "rgb(255, 255, 255)",
80-
"fqn_color": "rgb(255, 255, 255)",
81-
"fqn_type_color": "rgb(255, 160, 165)",
80+
"main_heading_color": "rgb(255, 255, 255)",
81+
"main_heading_type_color": "rgb(255, 160, 165)",
8282
"src_link_color": "rgb(57, 175, 215)",
8383
"sidebar_link_color": "rgb(83, 177, 219)",
8484
},
@@ -89,8 +89,8 @@ call-function: (
8989
"theme": "dark",
9090
"main_color": "rgb(221, 221, 221)",
9191
"title_color": "rgb(221, 221, 221)",
92-
"fqn_color": "rgb(221, 221, 221)",
93-
"fqn_type_color": "rgb(45, 191, 184)",
92+
"main_heading_color": "rgb(221, 221, 221)",
93+
"main_heading_type_color": "rgb(45, 191, 184)",
9494
"src_link_color": "rgb(210, 153, 29)",
9595
"sidebar_link_color": "rgb(253, 191, 53)",
9696
},
@@ -101,8 +101,8 @@ call-function: (
101101
"theme": "light",
102102
"main_color": "rgb(0, 0, 0)",
103103
"title_color": "rgb(0, 0, 0)",
104-
"fqn_color": "rgb(0, 0, 0)",
105-
"fqn_type_color": "rgb(173, 55, 138)",
104+
"main_heading_color": "rgb(0, 0, 0)",
105+
"main_heading_type_color": "rgb(173, 55, 138)",
106106
"src_link_color": "rgb(56, 115, 173)",
107107
"sidebar_link_color": "rgb(53, 109, 164)",
108108
},

‎tests/rustdoc-gui/headings.goml‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
// 14px 0.875rem
1414
goto: "file://" + |DOC_PATH| + "/test_docs/struct.HeavilyDocumentedStruct.html"
1515

16-
assert-css: ("h1.fqn", {"font-size": "24px"})
16+
assert-css: (".main-heading h1", {"font-size": "24px"})
1717

1818
assert-css: ("h2#top-doc-prose-title", {"font-size": "22px"})
1919
assert-css: ("h2#top-doc-prose-title", {"border-bottom-width": "1px"})
@@ -52,7 +52,7 @@ assert-css: ("h6#sub-sub-heading-for-struct-impl-item-doc", {"font-size": "14px"
5252

5353
goto: "file://" + |DOC_PATH| + "/test_docs/enum.HeavilyDocumentedEnum.html"
5454

55-
assert-css: ("h1.fqn", {"font-size": "24px"})
55+
assert-css: (".main-heading h1", {"font-size": "24px"})
5656

5757
assert-css: ("h2#top-doc-prose-title", {"font-size": "22px"})
5858
assert-css: ("h2#top-doc-prose-title", {"border-bottom-width": "1px"})
@@ -111,7 +111,7 @@ assert-css: ("//ul[@class='block mod']/preceding-sibling::h3", {"border-bottom-w
111111

112112
goto: "file://" + |DOC_PATH| + "/test_docs/union.HeavilyDocumentedUnion.html"
113113

114-
assert-css: ("h1.fqn", {"font-size": "24px"})
114+
assert-css: (".main-heading h1", {"font-size": "24px"})
115115

116116
assert-css: ("h2#top-doc-prose-title", {"font-size": "22px"})
117117
assert-css: ("h2#top-doc-prose-title", {"border-bottom-width": "1px"})
@@ -143,7 +143,7 @@ assert-css: ("h6#sub-heading-for-union-impl-item-doc", {"border-bottom-width": "
143143

144144
goto: "file://" + |DOC_PATH| + "/test_docs/macro.heavily_documented_macro.html"
145145

146-
assert-css: ("h1.fqn", {"font-size": "24px"})
146+
assert-css: (".main-heading h1", {"font-size": "24px"})
147147

148148
assert-css: ("h2#top-doc-prose-title", {"font-size": "22px"})
149149
assert-css: ("h2#top-doc-prose-title", {"border-bottom-width": "1px"})

‎tests/rustdoc-gui/search-result-go-to-first.goml‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,17 +3,17 @@
33
// First, we check that the first page doesn't have the string we're looking for to ensure
44
// that the feature is changing page as expected.
55
goto: "file://" + |DOC_PATH| + "/test_docs/index.html"
6-
assert-text-false: (".fqn", "Struct test_docs::Foo")
6+
assert-text-false: (".main-heading h1", "Struct test_docs::Foo")
77

88
// We now check that we land on the search result page if "go_to_first" isn't set.
99
goto: "file://" + |DOC_PATH| + "/test_docs/index.html?search=struct%3AFoo"
1010
// Waiting for the search results to appear...
1111
wait-for: "#search-tabs"
12-
assert-text-false: (".fqn", "Struct test_docs::Foo")
12+
assert-text-false: (".main-heading h1", "Struct test_docs::Foo")
1313
// Ensure that the search results are displayed, not the "normal" content.
1414
assert-css: ("#main-content", {"display": "none"})
1515

1616
// Now we can check that the feature is working as expected!
1717
goto: "file://" + |DOC_PATH| + "/test_docs/index.html?search=struct%3AFoo&go_to_first=true"
1818
// Waiting for the page to load...
19-
wait-for-text: (".fqn", "Struct test_docs::Foo")
19+
wait-for-text: (".main-heading h1", "Struct test_docs::Foo")

‎tests/rustdoc-gui/toggle-click-deadspace.goml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,4 @@ assert-attribute-false: (".impl-items .toggle", {"open": ""})
1212

1313
// Click the "Trait" part of "impl Trait" and verify it navigates.
1414
click: "#impl-Trait-for-Foo h3 a:first-of-type"
15-
assert-text: (".fqn", "Trait lib2::Trait")
15+
assert-text: (".main-heading h1", "Trait lib2::Trait")

0 commit comments

Comments
 (0)