Code review comment for lp:~geser/launchpadlib/toc

Revision history for this message
Michael Bienia (geser) wrote :

I've done the suggested fixes (r56).

And done also some unrelated fixes for other HTML errors (r57).
But there is still one error remaining: some <wadl:doc> elements already contain one (or more) <p>...</p> and the template for it adds another <p>...</p> around it which the W3C validator marks as an error. I didn't figure out how to only add a <p> if there is none (my XSLT/XPath knowledge is not good enough for it).

« Back to merge proposal