Attached is a patch that fixes most of the invalid XHTML in the modules.
If no one objects I'll fix the remaining bugs later on, so that this issue can be closed.

CommentFileSizeAuthor
xhtml.patch54.87 KBbylund
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Brian@brianpuccio.net’s picture

Brian@brianpuccio.net CreditAttribution: Brian@brianpuccio.net commented

Correct me if I'm wrong, but this only takes care of replacing all i or italic tags with em tags and all b or bold tags with strong tags, correct? Not any other invlaid XHTML?

I do think the patch should be applied (XHTML 1.1 compliance is a good thing, I've gotten my front page and every node page to validate (with a few bugs to work out) as XHTML 1.1) even though there is a lot more invalid code.

If a patch is present, shouldn't the status be set to patch so it gets added to the queue?

Dries’s picture

Dries CreditAttribution: Dries commented

Committed to the HEAD and DRUPAL-4-4 branch.

Anonymous’s picture

(not verified) CreditAttribution: commented