Fix clicking on toc entries for readtheorg

[alejandrogallo]

I guess this was a typo that did not get corrected!

[fniessen]

@alejandrogallo , thanks for this fix. However, I'm not aware of any problem. Could you describe what's expected, and what you get, without this patch?

Then, I'd apply it with pleasure!

Best regards...

[alejandrogallo]

Hi! well, if you open the inspector and click on a link of the toc then you get an error message saying that the variable nav_li_el is not found. Indeed in the js there is no such variable. I did some other changes, like for instance instead of a whole function for the activating of the ul's. At least in my testing with the current implementation when I click on a link on the toc then you'd have to click on it again so that it gets closed. I thought the intended behaviour should be that one fold opens and the rest close, which whould rather be done like this:

$(document).ready(function() {
    // these kids and their javascript...
    $("#text-table-of-contents a").click(function() {
        $("#text-table-of-contents a")
            .map((idx, a) => $(a).parent().removeClass("active"));
        $(this).parent().toggleClass("active");
    });
});

But I'm not sure this is the intended behaviour.

[fniessen]

Dear @alejandrogallo , thanks for the fix!