Skip to content

Commit 210c88f

Browse files
committed
rustdoc: clean up main.js and src-script.js
* Run the querySelector for the toggleLabel only once, and store the result. * Use querySelector to find the resizer and sidebar. * Add comments to main.js sections.
1 parent 0983438 commit 210c88f

File tree

2 files changed

+13
-6
lines changed

2 files changed

+13
-6
lines changed

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

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1273,6 +1273,7 @@ href="https://doc.rust-lang.org/${channel}/rustdoc/how-to-read-rustdoc.html\
12731273
searchState.setup();
12741274
}());
12751275

1276+
// This section handles sidebar resizing
12761277
(function() {
12771278
const sidebarButton = document.getElementById("sidebar-button");
12781279
if (sidebarButton) {
@@ -1283,8 +1284,8 @@ href="https://doc.rust-lang.org/${channel}/rustdoc/how-to-read-rustdoc.html\
12831284
});
12841285
}
12851286
let currentPointerId = null;
1286-
const resizer = document.getElementsByClassName("sidebar-resizer")[0];
1287-
const sidebar = document.getElementsByClassName("sidebar")[0];
1287+
const resizer = document.querySelector(".sidebar-resizer");
1288+
const sidebar = document.querySelector(".sidebar");
12881289
if (!resizer || !sidebar) {
12891290
return;
12901291
}
@@ -1379,6 +1380,7 @@ href="https://doc.rust-lang.org/${channel}/rustdoc/how-to-read-rustdoc.html\
13791380
resizer.addEventListener("pointerdown", initResize, false);
13801381
}());
13811382

1383+
// This section handles the copy button that appears next to the path breadcrumbs
13821384
(function() {
13831385
let reset_button_timeout = null;
13841386

src/librustdoc/html/static/js/src-script.js

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -71,17 +71,22 @@ function createDirEntry(elem, parent, fullPath, hasFoundFile) {
7171
return hasFoundFile;
7272
}
7373

74+
let toggleLabel;
75+
76+
function getToggleLabel() {
77+
toggleLabel = toggleLabel || document.querySelector("#src-sidebar-toggle button");
78+
return toggleLabel;
79+
}
80+
7481
window.rustdocCloseSourceSidebar = () => {
75-
const toggleLabel = document.querySelector("#src-sidebar-toggle button");
7682
removeClass(document.documentElement, "src-sidebar-expanded");
77-
toggleLabel.innerText = ">";
83+
getToggleLabel().innerText = ">";
7884
updateLocalStorage("source-sidebar-show", "false");
7985
};
8086

8187
window.rustdocShowSourceSidebar = () => {
82-
const toggleLabel = document.querySelector("#src-sidebar-toggle button");
8388
addClass(document.documentElement, "src-sidebar-expanded");
84-
toggleLabel.innerText = "<";
89+
getToggleLabel().innerText = "<";
8590
updateLocalStorage("source-sidebar-show", "true");
8691
};
8792

0 commit comments

Comments
 (0)