Diffs in the new 2.0 release are formatted in four columns, each 25% of the available width. This is happening because the COL elements aren't being output, so the CSS setting the column widths isn't being applied to the diff table.

CommentFileSizeAuthor
#4 contrib-5.diff.cols_.junyor.patch1.1 KBjunyor

Comments

junyor’s picture

junyor commented

It looks like this is only a problem when you preview changes on node edits.

moshe weitzman’s picture

moshe weitzman commented

Thanks Junyor. If you come up with a patch, I'll apply it.

junyor’s picture

junyor commented

I'll see what I can come up with. Does it make sense to call theme_diff_table() in diff_node_form_add_changes() instead of theme_table()? I'm not sure if there's a reason you didn't use theme_diff_table() there.

junyor’s picture

junyor commented
Status: Active » Needs review
StatusFileSize
new contrib-5.diff.cols_.junyor.patch1.1 KB

Assuming that's all that's needed, here's a patch that seems to do the trick.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Fixed

committed to 5. D6 handles this properly already.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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