Looks like the opening div is <div id="toc" class="toc">.

id="toc" is bad form because:

  1. A given id should only appear once on a page. A view could cause multiple nodes with TOCs to appear on a single page.
  2. This may cause confusion with the class of the same name. (I know, you refer to ids with a # in CSS, but would still help for them to be different.)

Suggested replacement is <div class="toc">.

Comments

AlexisWilke’s picture

AlexisWilke commented

It was used by the scrolling feature at some point, but it is not anymore. So indeed we can just get rid of it. Did you test the module without it already?

Thank you.
Alexis Wilke

aren cambre’s picture

aren cambre commented

Through Firebug on Firefox, I can delete the id with no ill effect. It indicates no CSS reference to #toc.

AlexisWilke’s picture

AlexisWilke commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.