Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.More than one level deep tables fail to validate as xhtml. Only list elements are allowed within lists.
It is easy to add the necessary <li> tags, and the first patch does that. Unfortunately that breaks list numbering (nested lists also count as elements). I don't know how to fix that in a cross-browser way.
The second patch replaces standard list numbering with counters (you can style those same as lists), and it works as supposed; however not in IE ... . IE doesn't support :before pseudo-class, so it doesn't display any numbers at all. It would be probably possible to generate those numbers within the script, or to implement some css hack to handle explorer, but I don't care that much ;)
Both patches done against 5.x-2.0 as this seems to be the most current version (cvs doesn't even have all files?).
You can find an example here: http://www.wrwrwr.org/node/11.
| Comment | File | Size | Author |
|---|---|---|---|
| tableofcontents.css_.patch | 459 bytes | wrwrwr | |
| tableofcontents.module.patch | 1.52 KB | wrwrwr |
Comments
Comment #1
deviantintegral commentedThanks for the patches. I believe I fixed these issues in #196056, can you test the -dev version and let me know if it works for you?
--Andrew
Comment #2
deviantintegral commentedI'm marking this as fixed as there haven't been any updates from the contributor. I haven't made the CSS changes since IE compatibility is rather important :). If there is an update to the module which works in IE, please create a patch and file a new issue. Thanks!
--Andrew
Comment #3
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.