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

Commit 5782f01

Browse files
borsbors
committed
Auto merge of rust-lang#87390 - notriddle:notriddle/rustdoc-headers-patch, r=GuillaumeGomez
Rustdoc accessibility: use real headers for doc items Part of rust-lang#87059 Partially reverts rust-lang#84703 Preview at: https://notriddle.com/notriddle-rustdoc-test/real-headers/std/index.html
2 parents 9c25eb7 + 76a3b60 commit 5782f01

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

79 files changed

+231
-218
lines changed

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

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1194,7 +1194,7 @@ fn notable_traits_decl(decl: &clean::FnDecl, cx: &Context<'_>) -> String {
11941194
if out.is_empty() {
11951195
write!(
11961196
&mut out,
1197-
"<h3 class=\"notable\">Notable traits for {}</h3>\
1197+
"<div class=\"notable\">Notable traits for {}</div>\
11981198
<code class=\"content\">",
11991199
impl_.for_.print(cx)
12001200
);
@@ -1350,15 +1350,15 @@ fn render_impl(
13501350
);
13511351
render_rightside(w, cx, item, containing_item);
13521352
write!(w, "<a href=\"#{}\" class=\"anchor\"></a>", id);
1353-
w.write_str("<code>");
1353+
w.write_str("<h4 class=\"code-header\">");
13541354
render_assoc_item(
13551355
w,
13561356
item,
13571357
link.anchor(source_id.as_ref().unwrap_or(&id)),
13581358
ItemType::Impl,
13591359
cx,
13601360
);
1361-
w.write_str("</code>");
1361+
w.write_str("</h4>");
13621362
w.write_str("</div>");
13631363
}
13641364
}
@@ -1371,7 +1371,7 @@ fn render_impl(
13711371
id, item_type, in_trait_class
13721372
);
13731373
write!(w, "<a href=\"#{}\" class=\"anchor\"></a>", id);
1374-
w.write_str("<code>");
1374+
w.write_str("<h4 class=\"code-header\">");
13751375
assoc_type(
13761376
w,
13771377
item,
@@ -1381,7 +1381,7 @@ fn render_impl(
13811381
"",
13821382
cx,
13831383
);
1384-
w.write_str("</code>");
1384+
w.write_str("</h4>");
13851385
w.write_str("</div>");
13861386
}
13871387
clean::AssocConstItem(ref ty, ref default) => {
@@ -1394,7 +1394,7 @@ fn render_impl(
13941394
);
13951395
render_rightside(w, cx, item, containing_item);
13961396
write!(w, "<a href=\"#{}\" class=\"anchor\"></a>", id);
1397-
w.write_str("<code>");
1397+
w.write_str("<h4 class=\"code-header\">");
13981398
assoc_const(
13991399
w,
14001400
item,
@@ -1404,15 +1404,15 @@ fn render_impl(
14041404
"",
14051405
cx,
14061406
);
1407-
w.write_str("</code>");
1407+
w.write_str("</h4>");
14081408
w.write_str("</div>");
14091409
}
14101410
clean::AssocTypeItem(ref bounds, ref default) => {
14111411
let source_id = format!("{}.{}", item_type, name);
14121412
let id = cx.derive_id(source_id.clone());
14131413
write!(w, "<div id=\"{}\" class=\"{}{}\">", id, item_type, in_trait_class,);
14141414
write!(w, "<a href=\"#{}\" class=\"anchor\"></a>", id);
1415-
w.write_str("<code>");
1415+
w.write_str("<h4 class=\"code-header\">");
14161416
assoc_type(
14171417
w,
14181418
item,
@@ -1422,7 +1422,7 @@ fn render_impl(
14221422
"",
14231423
cx,
14241424
);
1425-
w.write_str("</code>");
1425+
w.write_str("</h4>");
14261426
w.write_str("</div>");
14271427
}
14281428
clean::StrippedItem(..) => return,
@@ -1613,7 +1613,7 @@ pub(crate) fn render_impl_summary(
16131613
write!(w, "<div id=\"{}\" class=\"impl has-srclink\"{}>", id, aliases);
16141614
render_rightside(w, cx, &i.impl_item, containing_item);
16151615
write!(w, "<a href=\"#{}\" class=\"anchor\"></a>", id);
1616-
write!(w, "<code class=\"in-band\">");
1616+
write!(w, "<h3 class=\"code-header in-band\">");
16171617

16181618
if let Some(use_absolute) = use_absolute {
16191619
write!(w, "{}", i.inner_impl().print(use_absolute, cx));
@@ -1629,7 +1629,7 @@ pub(crate) fn render_impl_summary(
16291629
} else {
16301630
write!(w, "{}", i.inner_impl().print(false, cx));
16311631
}
1632-
write!(w, "</code>");
1632+
write!(w, "</h3>");
16331633

16341634
let is_trait = i.inner_impl().trait_.is_some();
16351635
if is_trait {

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -621,9 +621,9 @@ fn item_trait(w: &mut Buffer, cx: &Context<'_>, it: &clean::Item, t: &clean::Tra
621621
render_stability_since(w, m, t, cx.tcx());
622622
write_srclink(cx, m, w);
623623
write!(w, "</div>");
624-
write!(w, "<code>");
624+
write!(w, "<h4 class=\"code-header\">");
625625
render_assoc_item(w, m, AssocItemLink::Anchor(Some(&id)), ItemType::Impl, cx);
626-
w.write_str("</code>");
626+
w.write_str("</h4>");
627627
w.write_str("</div>");
628628
if toggled {
629629
write!(w, "</summary>");

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

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -146,6 +146,13 @@ h1.fqn > .in-band > a:hover {
146146
h2, h3, h4 {
147147
border-bottom: 1px solid;
148148
}
149+
h3.code-header, h4.code-header {
150+
font-size: 1em;
151+
font-weight: 600;
152+
border: none;
153+
padding: 0;
154+
margin: 0;
155+
}
149156
.impl,
150157
.impl-items .method,
151158
.methods .method,
@@ -233,7 +240,7 @@ details:not(.rustdoc-toggle) summary {
233240
margin-bottom: .6em;
234241
}
235242

236-
code, pre, a.test-arrow {
243+
code, pre, a.test-arrow, .code-header {
237244
font-family: "Source Code Pro", monospace;
238245
}
239246
.docblock code, .docblock-short code {
@@ -521,7 +528,7 @@ nav.sub {
521528
font-weight: normal;
522529
}
523530

524-
.method > code, .trait-impl > code, .invisible > code {
531+
.method > .code-header, .trait-impl > .code-header, .invisible > .code-header {
525532
max-width: calc(100% - 41px);
526533
display: block;
527534
}
@@ -537,7 +544,7 @@ nav.sub {
537544
padding: 0px;
538545
}
539546

540-
.in-band > code {
547+
.in-band > code, .in-band > .code-header {
541548
display: inline-block;
542549
}
543550

@@ -743,7 +750,7 @@ a {
743750
}
744751

745752
.invisible > .srclink,
746-
.method > code + .srclink {
753+
.method > .code-header + .srclink {
747754
position: absolute;
748755
top: 0;
749756
right: 0;
@@ -1104,7 +1111,7 @@ a.test-arrow:hover{
11041111
left: -10px;
11051112
}
11061113

1107-
:target > code {
1114+
:target > code, :target > .code-header {
11081115
opacity: 1;
11091116
}
11101117

‎src/librustdoc/html/static/css/themes/ayu.css‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -334,7 +334,7 @@ a.test-arrow:hover {
334334
color: #999;
335335
}
336336

337-
:target, :target * {
337+
:target, :target > * {
338338
background: rgba(255, 236, 164, 0.06);
339339
}
340340

‎src/librustdoc/html/static/css/themes/dark.css‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -282,7 +282,7 @@ a.test-arrow:hover{
282282
color: #999;
283283
}
284284

285-
:target, :target * {
285+
:target, :target > * {
286286
background-color: #494a3d;
287287
}
288288

‎src/librustdoc/html/static/css/themes/light.css‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -274,7 +274,7 @@ a.test-arrow:hover{
274274
color: #999;
275275
}
276276

277-
:target, :target * {
277+
:target, :target > * {
278278
background: #FDFFD3;
279279
}
280280

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

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -706,8 +706,9 @@ function hideThemeButtonState() {
706706
}
707707
}
708708

709-
var code = document.createElement("code");
709+
var code = document.createElement("h3");
710710
code.innerHTML = struct.text;
711+
addClass(code, "code-header");
711712
addClass(code, "in-band");
712713

713714
onEachLazy(code.getElementsByTagName("a"), function(elem) {

‎src/test/rustdoc-gui/font-weight.goml‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@ goto: file://|DOC_PATH|/lib2/struct.Foo.html
22
// This test checks that the font weight is correctly applied.
33
assert-css: ("//*[@class='docblock type-decl']//a[text()='Alias']", {"font-weight": "400"})
44
assert-css: ("//*[@class='structfield small-section-header']//a[text()='Alias']", {"font-weight": "400"})
5-
assert-css: ("#method\.a_method > code", {"font-weight": "600"})
6-
assert-css: ("#associatedtype\.X > code", {"font-weight": "600"})
7-
assert-css: ("#associatedconstant\.Y > code", {"font-weight": "600"})
5+
assert-css: ("#method\.a_method > .code-header", {"font-weight": "600"})
6+
assert-css: ("#associatedtype\.X > .code-header", {"font-weight": "600"})
7+
assert-css: ("#associatedconstant\.Y > .code-header", {"font-weight": "600"})
88

99
goto: file://|DOC_PATH|/test_docs/type.SomeType.html
1010
assert-css: (".top-doc .docblock p", {"font-weight": "400"}, ALL)

‎src/test/rustdoc-gui/implementors.goml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,9 @@ assert-count: ("#implementors-list > .impl", 2)
88
assert: ("#implementors-list > .impl:nth-child(1) > a.anchor")
99
assert-attribute: ("#implementors-list > .impl:nth-child(1)", {"id": "impl-Whatever"})
1010
assert-attribute: ("#implementors-list > .impl:nth-child(1) > a.anchor", {"href": "#impl-Whatever"})
11-
assert: "#implementors-list > .impl:nth-child(1) > code.in-band"
11+
assert: "#implementors-list > .impl:nth-child(1) > .code-header.in-band"
1212

1313
assert: ("#implementors-list > .impl:nth-child(2) > a.anchor")
1414
assert-attribute: ("#implementors-list > .impl:nth-child(2)", {"id": "impl-Whatever-1"})
1515
assert-attribute: ("#implementors-list > .impl:nth-child(2) > a.anchor", {"href": "#impl-Whatever-1"})
16-
assert: "#implementors-list > .impl:nth-child(2) > code.in-band"
16+
assert: "#implementors-list > .impl:nth-child(2) > .code-header.in-band"

‎src/test/rustdoc/assoc-consts.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ pub trait Foo {
1313
pub struct Bar;
1414

1515
impl Foo for Bar {
16-
// @has assoc_consts/struct.Bar.html '//code' 'impl Foo for Bar'
16+
// @has assoc_consts/struct.Bar.html '//h3[@class="code-header in-band"]' 'impl Foo for Bar'
1717
// @has - '//*[@id="associatedconstant.FOO"]' 'const FOO: usize'
1818
const FOO: usize = 12;
1919
// @has - '//*[@id="associatedconstant.FOO_NO_DEFAULT"]' 'const FOO_NO_DEFAULT: bool'
@@ -77,7 +77,7 @@ pub trait Qux {
7777
const QUX_DEFAULT2: u32 = 3;
7878
}
7979

80-
// @has assoc_consts/struct.Bar.html '//code' 'impl Qux for Bar'
80+
// @has assoc_consts/struct.Bar.html '//h3[@class="code-header in-band"]' 'impl Qux for Bar'
8181
impl Qux for Bar {
8282
// @has - '//*[@id="associatedconstant.QUX0"]' 'const QUX0: u8'
8383
// @has - '//*[@class="docblock"]' "Docs for QUX0 in trait."

0 commit comments

Comments
 (0)