Attached is a patch that alters the table-generation code to add 'odd'/'even' CSS classes, so Markdown tables get alternating row styling (like other Drupal tables).

CommentFileSizeAuthor
#5 markdown.php_listitemrowalteration.patch971 byteslolmaus
markdown.php_tablerowalternation.patch1.02 KBsmokris

Comments

lolmaus’s picture

lolmaus commented

Works for me! Great patch!

PS Clear cache to see the result.

lolmaus’s picture

lolmaus commented
Status: Needs review » Reviewed & tested by the community
beeradb’s picture

beeradb commented
Status: Reviewed & tested by the community » Needs work

Please update this to match the standards outlined in http://drupal.org/coding-standards. In particular, I see issues with whitespace around string concatenations and the formatting of if/else blocks.

justin2pin’s picture

justin2pin commented
Status: Needs work » Closed (won't fix)

I don't think patching a 3rd-party library is the way we want to go with this. I'd suggest either (a) emailing Michel Fortin (http://michelf.com/projects/php-markdown/) to see if he'd be interested in this modification for Markdown, after which I'd be happy to include his changes in another module release; or (b) moving in the direction sreynen outlines here: http://drupal.org/node/515048#comment-2906948.

lolmaus’s picture

lolmaus commented
StatusFileSize
new markdown.php_listitemrowalteration.patch971 bytes

And here comes a patch for list item row alteration!