From 58453418266a7367fae5e9e4a5f78614ad5ff3ce Mon Sep 17 00:00:00 2001 From: David Stansby Date: Sun, 28 Sep 2025 08:10:58 +0200 Subject: [PATCH 1/2] Fix error when back-to-top button is disabled --- src/pydata_sphinx_theme/assets/scripts/bootstrap.js | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/src/pydata_sphinx_theme/assets/scripts/bootstrap.js b/src/pydata_sphinx_theme/assets/scripts/bootstrap.js index 505a65eff8..af3ba6fae3 100644 --- a/src/pydata_sphinx_theme/assets/scripts/bootstrap.js +++ b/src/pydata_sphinx_theme/assets/scripts/bootstrap.js @@ -14,7 +14,7 @@ import { documentReady } from "./mixin"; */ function TriggerTooltip() { var tooltipTriggerList = [].slice.call( - document.querySelectorAll('[data-bs-toggle="tooltip"]'), + document.querySelectorAll('[data-bs-toggle="tooltip"]') ); tooltipTriggerList.map(function (tooltipTriggerEl) { return new bootstrap.Tooltip(tooltipTriggerEl, { @@ -28,10 +28,13 @@ function TriggerTooltip() { */ function backToTop() { var btn = document.getElementById("pst-back-to-top"); - btn.addEventListener("click", function () { - document.body.scrollTop = 0; - document.documentElement.scrollTop = 0; - }); + // Config allows button to be disabled, so check it exists + if (btn) { + btn.addEventListener("click", function () { + document.body.scrollTop = 0; + document.documentElement.scrollTop = 0; + }); + } } function showBackToTop() { From 7f423b1164ec65a8af0fecbe4ef4ee697fc81880 Mon Sep 17 00:00:00 2001 From: "pre-commit-ci[bot]" <66853113+pre-commit-ci[bot]@users.noreply.github.com> Date: Sun, 28 Sep 2025 06:12:39 +0000 Subject: [PATCH 2/2] [pre-commit.ci] Automatic linting and formatting fixes --- src/pydata_sphinx_theme/assets/scripts/bootstrap.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/pydata_sphinx_theme/assets/scripts/bootstrap.js b/src/pydata_sphinx_theme/assets/scripts/bootstrap.js index af3ba6fae3..d3cb2de87d 100644 --- a/src/pydata_sphinx_theme/assets/scripts/bootstrap.js +++ b/src/pydata_sphinx_theme/assets/scripts/bootstrap.js @@ -14,7 +14,7 @@ import { documentReady } from "./mixin"; */ function TriggerTooltip() { var tooltipTriggerList = [].slice.call( - document.querySelectorAll('[data-bs-toggle="tooltip"]') + document.querySelectorAll('[data-bs-toggle="tooltip"]'), ); tooltipTriggerList.map(function (tooltipTriggerEl) { return new bootstrap.Tooltip(tooltipTriggerEl, {