Here's a screenshot and some code to with from that node. Test by creating a new node, putting this text in the body and setting the Full HTML filter.

CommentFileSizeAuthor
#5 bartik-tbody-th.patch926 bytesJeff Burnz
#5 bartik-tbody-th-IE9.png79.96 KBJeff Burnz
#3 better-table.png390.21 KBJacine
test-code.txt1.66 KBJacine
tables-with-headers-in-tbody.png430.75 KBJacine
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jeff Burnz’s picture

Jeff Burnz CreditAttribution: Jeff Burnz commented

Any chance of a patch Jacine :)

Jacine’s picture

Jacine
she/her
Location New York
CreditAttribution: Jacine commented

Hey, yes... I just have a major issue with these tables, period. I think they are really ugly and don't fit with the design at all, and when viewed this way, they annoy me 10x more. I'm going to try and track down Jen to see what she thinks.

Jacine’s picture

Jacine
she/her
Location New York
CreditAttribution: Jacine commented
FileSize
better-table.png390.21 KB

I'd like to see something like this TBH.

Jeff Burnz’s picture

Jeff Burnz CreditAttribution: Jeff Burnz commented

Nik Lewis did the current design, which was a big improvement on what we had, I know we discussed th in tbody but looks like that never happened.

I think we're a bit late in the game for a design change (just saying like), but I would think that no vertical borders won't work so well with real tabular data (like a spreadsheet).

When I look at the screenshot what I see is a document list (as in if you had this particular content and wanted it to look this way, I would use a DL). Do you sort of get what I mean?

Jeff Burnz’s picture

Jeff Burnz CreditAttribution: Jeff Burnz commented
Status: Active » Needs review
FileSize
bartik-tbody-th-IE9.png79.96 KB
bartik-tbody-th.patch926 bytes

Perhaps we can do a quickfix and make them consistent with thead th, but use vertical-align top?

bartik-tbody-th-IE9.png

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented

Looks visually good to me, patch still applies. Much better than without :)

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Needs review » Reviewed & tested by the community

This looks great, especially in conjunction with #967132: The forum colors are out of sync with the theme colors.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

Status: Fixed » Closed (fixed)

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