Rollup merge of #92830 - jsha:style-cleanups, r=GuillaumeGomez

Rustdoc style cleanups

 - Make "since" version numbers grey again (regressed in #92602).
 - Remove unneeded selectors for when crate filter dropdown is a
   sibling of search-input.
 - Crate filter dropdown doesn't need to be 100% width on mobile.
 - Only build crate filter dropdown when there is more than one crate.
 - Remove unused addCrateDropdown

 Demo: https://rustdoc.crud.net/jsha/style-cleanups/std/string/struct.String.html

 r? `@GuillaumeGomez`
This commit is contained in:
Matthias Krüger 2022-01-18 04:42:02 +01:00 committed by GitHub
commit deee6f770f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
10 changed files with 62 additions and 41 deletions

View File

@ -944,11 +944,6 @@ h2.small-section-header > .anchor {
width: 100%; width: 100%;
} }
#crate-search + .search-input {
border-radius: 0 1px 1px 0;
width: calc(100% - 32px);
}
.search-input:focus { .search-input:focus {
border-radius: 2px; border-radius: 2px;
border: 0; border: 0;
@ -2081,7 +2076,6 @@ details.rustdoc-toggle[open] > summary.hideme::after {
} }
#crate-search { #crate-search {
width: 100%;
border-radius: 4px; border-radius: 4px;
border: 0; border: 0;
} }

View File

@ -299,7 +299,8 @@ details.undocumented > summary::before {
border-color: #5c6773; border-color: #5c6773;
} }
.since { .rightside,
.out-of-band {
color: grey; color: grey;
} }

View File

@ -256,7 +256,8 @@ details.undocumented > summary::before {
background: rgba(0,0,0,0); background: rgba(0,0,0,0);
} }
.since { .rightside,
.out-of-band {
color: grey; color: grey;
} }

View File

@ -243,6 +243,11 @@ details.undocumented > summary::before {
border-color: #bfbfbf; border-color: #bfbfbf;
} }
.rightside,
.out-of-band {
color: grey;
}
.result-name .primitive > i, .result-name .keyword > i { .result-name .primitive > i, .result-name .keyword > i {
color: black; color: black;
} }

View File

@ -288,9 +288,6 @@ function hideThemeButtonState() {
loadSearch(); loadSearch();
} }
// `crates{version}.js` should always be loaded before this script, so we can use it
// safely.
searchState.addCrateDropdown(window.ALL_CRATES);
var params = searchState.getQueryStringParams(); var params = searchState.getQueryStringParams();
if (params.search !== undefined) { if (params.search !== undefined) {
var search = searchState.outputElement(); var search = searchState.outputElement();
@ -300,30 +297,6 @@ function hideThemeButtonState() {
loadSearch(); loadSearch();
} }
}, },
addCrateDropdown: function(crates) {
var elem = document.getElementById("crate-search");
if (!elem) {
return;
}
var savedCrate = getSettingValue("saved-filter-crate");
for (var i = 0, len = crates.length; i < len; ++i) {
var option = document.createElement("option");
option.value = crates[i];
option.innerText = crates[i];
elem.appendChild(option);
// Set the crate filter from saved storage, if the current page has the saved crate
// filter.
//
// If not, ignore the crate filter -- we want to support filtering for crates on
// sites like doc.rust-lang.org where the crates may differ from page to page while
// on the
// same domain.
if (crates[i] === savedCrate) {
elem.value = savedCrate;
}
}
},
}; };
function getPageId() { function getPageId() {

View File

@ -1126,15 +1126,18 @@ window.initSearch = function(rawSearchIndex) {
} }
} }
let crates = `<select id="crate-search"><option value="All crates">All crates</option>`; let crates = "";
for (let c of window.ALL_CRATES) { if (window.ALL_CRATES.length > 1) {
crates += `<option value="${c}" ${c == filterCrates && "selected"}>${c}</option>`; crates = ` in <select id="crate-search"><option value="All crates">All crates</option>`;
for (let c of window.ALL_CRATES) {
crates += `<option value="${c}" ${c == filterCrates && "selected"}>${c}</option>`;
}
crates += `</select>`;
} }
crates += `</select>`;
var output = `<div id="search-settings"> var output = `<div id="search-settings">
<h1 class="search-results-title">Results for ${escape(query.query)} ` + <h1 class="search-results-title">Results for ${escape(query.query)} ` +
(query.type ? " (type: " + escape(query.type) + ")" : "") + "</h1>" + (query.type ? " (type: " + escape(query.type) + ")" : "") + "</h1>" +
` in ${crates} ` + crates +
`</div><div id="titles">` + `</div><div id="titles">` +
makeTabHeader(0, "In Names", ret_others[1]) + makeTabHeader(0, "In Names", ret_others[1]) +
makeTabHeader(1, "In Parameters", ret_in_args[1]) + makeTabHeader(1, "In Parameters", ret_in_args[1]) +
@ -1148,7 +1151,10 @@ window.initSearch = function(rawSearchIndex) {
resultsElem.appendChild(ret_returned[0]); resultsElem.appendChild(ret_returned[0]);
search.innerHTML = output; search.innerHTML = output;
document.getElementById("crate-search").addEventListener("input", updateCrate); let crateSearch = document.getElementById("crate-search");
if (crateSearch) {
crateSearch.addEventListener("input", updateCrate);
}
search.appendChild(resultsElem); search.appendChild(resultsElem);
// Reset focused elements. // Reset focused elements.
searchState.focusedByTab = [null, null, null]; searchState.focusedByTab = [null, null, null];

View File

@ -154,3 +154,16 @@ assert-css: ("h2#top-doc-prose-title", {"font-size": "20.8px"})
assert-css: ("h2#top-doc-prose-title", {"border-bottom-width": "1px"}) assert-css: ("h2#top-doc-prose-title", {"border-bottom-width": "1px"})
assert-css: ("h3#top-doc-prose-sub-heading", {"font-size": "18.4px"}) assert-css: ("h3#top-doc-prose-sub-heading", {"font-size": "18.4px"})
assert-css: ("h3#top-doc-prose-sub-heading", {"border-bottom-width": "1px"}) assert-css: ("h3#top-doc-prose-sub-heading", {"border-bottom-width": "1px"})
goto: file://|DOC_PATH|/staged_api/struct.Foo.html
show-text: true
local-storage: {"rustdoc-theme": "light", "rustdoc-use-system-theme": "false"}
assert-css: (".since", {"color": "rgb(128, 128, 128)"})
local-storage: {"rustdoc-theme": "dark", "rustdoc-use-system-theme": "false"}
reload:
assert-css: (".since", {"color": "rgb(128, 128, 128)"})
local-storage: {"rustdoc-theme": "ayu", "rustdoc-use-system-theme": "false"}
reload:
assert-css: (".since", {"color": "rgb(128, 128, 128)"})

View File

@ -0,0 +1,7 @@
# This file is automatically @generated by Cargo.
# It is not intended for manual editing.
version = 3
[[package]]
name = "staged_api"
version = "0.1.0"

View File

@ -0,0 +1,11 @@
[package]
name = "staged_api"
version = "0.1.0"
edition = "2021"
[lib]
path = "lib.rs"
[features]
default = ["some_feature"]
some_feature = []

View File

@ -0,0 +1,10 @@
#![feature(staged_api)]
#![stable(feature = "some_feature", since = "1.3.5")]
#[stable(feature = "some_feature", since = "1.3.5")]
pub struct Foo {}
impl Foo {
#[stable(feature = "some_feature", since = "1.3.5")]
pub fn bar() {}
}