View attached screenshots to see the difference.

Although views is a contrib module I think Zen should be 'nice' and define for it as well.

CommentFileSizeAuthor
#1 342004_1_table_width.patch396 bytesamitaibu
zen.png16.87 KBamitaibu
garland.png17.05 KBamitaibu
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

amitaibu’s picture

Title: Make tables and Views table width = 100% » Make tables width = 100%
Status: Active » Needs review
FileSize
396 bytes

sorry, views has nothing to do with it. Patch attached.

kmonty’s picture

This is something I set on basically all of my websites. The only drawback could be potentially confusing someone who just wants a narrow table centered on a page in the middle of an article, or aligned to the right/left/etc of a page.

However, Zen's point is to be light started theme and not really presume how you want to style your content. This might not be appropriate.

amitaibu’s picture

Even if this is the case it's worth having a placeholder, i.e.

table
{
/*width: 100 */ /* Define the width of a table. */
}
yoroy’s picture

I like this because autosized tables look rather cramped by default. Going for 100% width by default would actually make most tables a bit more readable I think. The point #2 makes against this is a good one but I wouldn't mind Zen providing a "better" default here.

+1, can't really rtbc this because I didn't apply the actual patch.

SeanBannister’s picture

+1 I always make this change as well, while Zen is designed to be light weight there would be few people who wouldn't make this change.

JohnAlbin’s picture

Title: Make tables width = 100% » Add css hint to prevent cramped-looking tables
Component: Subtheme: Zen Classic » Code
Status: Needs review » Fixed

I went with the placeholder /* width: 100%; */ /* Prevent cramped-looking tables */

Thanks, Amitai!

Status: Fixed » Closed (fixed)

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