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

Comments

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
commented
Title: Make tables and Views table width = 100% » Make tables width = 100%
Status: Active » Needs review
StatusFileSize
new 342004_1_table_width.patch396 bytes

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

kmonty’s picture

kmonty
He/Him
Location Berlin, DE + SF, CA
commented

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

amitaibu
Primary language Hebrew
Location Israel
commented

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

yoroy commented

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

SeanBannister commented

+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

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
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.