Quoting Jonathan Corbet (2024-01-23 18:33:52-03:00)
>Gustavo Sousa <gustavo.sousa@xxxxxxxxx> writes:
>
>> When on the documentation home page, there won't be any ".current"
>> element since no entry from the TOC was selected yet. That results in a
>> javascript error. Fix that by only trying to set the scrollTop if we
>> have matches for current entries.
>>
>> Signed-off-by: Gustavo Sousa <gustavo.sousa@xxxxxxxxx>
>> ---
>> Documentation/sphinx/templates/kernel-toc.html | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/Documentation/sphinx/templates/kernel-toc.html b/Documentation/sphinx/templates/kernel-toc.html
>> index b58efa99df52..41f1efbe64bb 100644
>> --- a/Documentation/sphinx/templates/kernel-toc.html
>> +++ b/Documentation/sphinx/templates/kernel-toc.html
>> @@ -12,5 +12,7 @@
>> <script type="text/javascript"> <!--
>> var sbar = document.getElementsByClassName("sphinxsidebar")[0];
>> let currents = document.getElementsByClassName("current")
>> - sbar.scrollTop = currents[currents.length - 1].offsetTop;
>> + if (currents.length) {
>> + sbar.scrollTop = currents[currents.length - 1].offsetTop;
>> + }
>> --> </script>
>
>Interesting, that's definitely a bug. Did you stumble across this in
>the console, or did it manifest in some other way for you?
I just saw the error in the console :-)
Fortunately, the error is currently harmless, but it could become a
problem in the future if we were to put more stuff after the offending
line.
>
>Either way, I've applied this, thanks.
Thanks!
--
Gustavo Sousa
