Currently, the resource toolbox initialiser (specifically, _setup_for_resource) wastes time by mucking about fetching show/hide icons and then fetching their section ancestor in order to check whether it's hidden, then pointlessly setting the show/hide icon's cursor value to "auto" if the section is hidden. This achieves nothing except wasting time - it's not even doing what it was intended to, since cursor:auto on an a tag (which the show/hide icon is) results in the hand icon anyway!

This should be done using CSS instead (setting it to cursor:default, which actually achieves the stated goal ("Disable" show/hide icons (change cursor to not look clickable) if section is hidden).