languages/rust
languages/rust
1
0
Fork 0
mirror of https://github.com/rust-lang/rust synced 2024-10-14 12:33:57 +00:00
Code Issues Packages Projects Releases Wiki Activity

rustdoc: make item links consistently use title="{shortty} {path}"

Browse source 
The ordering in item tables was flipped in 3030cbea95, making it
inconsistent with the ordering in method signatures.

Compare these:

c8e6a9e8b6/src/librustdoc/html/render/print_item.rs (L455-L459)

c8e6a9e8b6/src/librustdoc/html/format.rs (L903-L908)
This commit is contained in:
Michael Howell 2023-01-23 14:31:35 -07:00
parent c8e6a9e8b6
commit 57ca36861d
7 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/librustdoc/html/render/print_item.rs
View file

@ -452,7 +452,7 @@ fn cmp(
stab = stab.unwrap_or_default(),
unsafety_flag = unsafety_flag,
href = item_path(myitem.type_(), myitem.name.unwrap().as_str()),
title = [full_path(cx, myitem), myitem.type_().to_string()]
title = [myitem.type_().to_string(), full_path(cx, myitem)]
.iter()
.filter_map(|s| if !s.is_empty() { Some(s.as_str()) } else { None })
.collect::<Vec<_>>()

4
tests/rustdoc-gui/unsafe-fn.goml
View file

@ -4,8 +4,8 @@ goto: "file://" + |DOC_PATH| + "/test_docs/index.html"
show-text: true
compare-elements-property: (
"//a[@title='test_docs::safe_fn fn']/..",
"//a[@title='test_docs::unsafe_fn fn']/..",
"//a[@title='fn test_docs::safe_fn']/..",
"//a[@title='fn test_docs::unsafe_fn']/..",
["clientHeight"]
)

2
tests/rustdoc/issue-83375-multiple-mods-w-same-name-doc-inline-last-item.rs
View file

@ -11,6 +11,6 @@ pub mod prelude {
#[doc(inline)]
pub use sub::*;
// @count foo/index.html '//a[@class="mod"][@title="foo::prelude mod"]' 1
// @count foo/index.html '//a[@class="mod"][@title="mod foo::prelude"]' 1
// @count foo/prelude/index.html '//div[@class="item-row"]' 0
pub mod prelude {}

2
tests/rustdoc/issue-83375-multiple-mods-w-same-name-doc-inline.rs
View file

@ -8,7 +8,7 @@ pub mod prelude {
}
}
// @count foo/index.html '//a[@class="mod"][@title="foo::prelude mod"]' 1
// @count foo/index.html '//a[@class="mod"][@title="mod foo::prelude"]' 1
// @count foo/prelude/index.html '//div[@class="item-row"]' 0
pub mod prelude {}

2
tests/rustdoc/issue-99221-multiple-structs-w-same-name.rs
View file

@ -9,6 +9,6 @@
pub use issue_99221_aux::*;
// @count foo/index.html '//a[@class="struct"][@title="foo::Print struct"]' 1
// @count foo/index.html '//a[@class="struct"][@title="struct foo::Print"]' 1
pub struct Print;

2
tests/rustdoc/issue-99734-multiple-foreigns-w-same-name.rs
View file

@ -9,7 +9,7 @@
pub use issue_99734_aux::*;
// @count foo/index.html '//a[@class="fn"][@title="foo::main fn"]' 1
// @count foo/index.html '//a[@class="fn"][@title="fn foo::main"]' 1
extern "C" {
pub fn main() -> std::ffi::c_int;

2
tests/rustdoc/issue-99734-multiple-mods-w-same-name.rs
View file

@ -9,6 +9,6 @@
pub use issue_99734_aux::*;
// @count foo/index.html '//a[@class="mod"][@title="foo::task mod"]' 1
// @count foo/index.html '//a[@class="mod"][@title="mod foo::task"]' 1
pub mod task {}