Skip to content

Commit 3375b05

Browse files
Fix confusion in theme picker functions
1 parent 7858dc2 commit 3375b05

File tree

2 files changed

+14
-14
lines changed

2 files changed

+14
-14
lines changed

src/librustdoc/html/render.rs

+8-8
Original file line numberDiff line numberDiff line change
@@ -876,22 +876,22 @@ r#"var themes = document.getElementById("theme-choices");
876876
var themePicker = document.getElementById("theme-picker");
877877
878878
function showThemeButtonState() {{
879-
themes.style.display = "none";
880-
themePicker.style.borderBottomRightRadius = "3px";
881-
themePicker.style.borderBottomLeftRadius = "3px";
882-
}}
883-
884-
function hideThemeButtonState() {{
885879
themes.style.display = "block";
886880
themePicker.style.borderBottomRightRadius = "0";
887881
themePicker.style.borderBottomLeftRadius = "0";
888882
}}
889883
884+
function hideThemeButtonState() {{
885+
themes.style.display = "none";
886+
themePicker.style.borderBottomRightRadius = "3px";
887+
themePicker.style.borderBottomLeftRadius = "3px";
888+
}}
889+
890890
function switchThemeButtonState() {{
891891
if (themes.style.display === "block") {{
892-
showThemeButtonState();
893-
}} else {{
894892
hideThemeButtonState();
893+
}} else {{
894+
showThemeButtonState();
895895
}}
896896
}};
897897

src/librustdoc/html/static/main.js

+6-6
Original file line numberDiff line numberDiff line change
@@ -105,9 +105,9 @@ if (!DOMTokenList.prototype.remove) {
105105
sidebar.appendChild(div);
106106
}
107107
}
108-
var themePicker = document.getElementsByClassName("theme-picker");
109-
if (themePicker && themePicker.length > 0) {
110-
themePicker[0].style.display = "none";
108+
var themePickers = document.getElementsByClassName("theme-picker");
109+
if (themePickers && themePickers.length > 0) {
110+
themePickers[0].style.display = "none";
111111
}
112112
}
113113

@@ -123,9 +123,9 @@ if (!DOMTokenList.prototype.remove) {
123123
filler.remove();
124124
}
125125
document.getElementsByTagName("body")[0].style.marginTop = "";
126-
var themePicker = document.getElementsByClassName("theme-picker");
127-
if (themePicker && themePicker.length > 0) {
128-
themePicker[0].style.display = null;
126+
var themePickers = document.getElementsByClassName("theme-picker");
127+
if (themePickers && themePickers.length > 0) {
128+
themePickers[0].style.display = null;
129129
}
130130
}
131131

0 commit comments

Comments
 (0)