As the title says, theme_casetracker_case_form_common should zero pad the last row of the "Assign to" table, otherwise there is a chance it wont be equally many columns in the table body as there is in the table head.

Before:
Before

After:
After

CommentFileSizeAuthor
After17.67 KBlogaritmisk
Before17.77 KBlogaritmisk
theme_casetracker_case_form_common_zero_pad_assign_to-1.patch521 byteslogaritmisk

Comments

simon georges’s picture

simon georges commented
Status: Active » Needs review

Changing status, as there is a patch.

skyredwang’s picture

skyredwang
Location Shanghai
commented
Status: Needs review » Closed (won't fix)

I don't know why this is necessary

logaritmisk’s picture

logaritmisk commented

From http://www.w3.org/TR/html401/struct/tables.html, Calculating the number of columns in a table:

Otherwise, if the TABLE element contains no COLGROUP or COL elements, user agents should base the number of columns on what is required by the rows. The number of columns is equal to the number of columns required by the row with the most columns, including cells that span multiple columns. For any row that has fewer than this number of columns, the end of that row should be padded with empty cells. The "end" of a row depends on the table directionality.

skyredwang’s picture

skyredwang
Location Shanghai
commented
Status: Closed (won't fix) » Reviewed & tested by the community

Good documentation! Great suggestion! I learned a new thing!

skyredwang’s picture

skyredwang
Location Shanghai
commented
Status: Reviewed & tested by the community » Closed (fixed)

committed http://drupalcode.org/project/casetracker.git/commit/fcad6a8