This issue does not exist in the 4.x-dev version because the code has changed. So please don't reset the version to 4.x-dev with a port-back to v3 tag. This only exists in version 3. (currently)

By default Omega adds "border: 0;" to the "table" element. This makes it impossible for beginners to override the border in the WYSIWYG scenario. When using a WYSIWYG the border is added via border="1" and not through style="border: 1px". Therefore the default Omega border always overrides the user entered value.

I know that it's possible for someone to put style="border: 5px;" in the WYSIWYG and make it work. However, most of the clients I (and probably most people) build websites for don't have CSS knowledge.

So to keep this user friendly I suggest we remove the borders on tables as a default.

CommentFileSizeAuthor
#1 remove_table_borders-1751362-1.patch251 bytesSteven Brown
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Steven Brown’s picture

Here's the patch to fix this.

Steven Brown’s picture

Status: Active » Needs review

Sorry I was in a bit of a hurry there.

Cellar Door’s picture

Assigned: Unassigned » himerus

We'll have to look at how this will effect the existing sites in production. Seems small enough not to matter but now that we've got so many sites small edits to the base css like this need to be done carefully. I'll talk w/ himerus to see if it can go into the 3.2 update.

Steven Brown’s picture

Thank you for the reply. I couldn't agree with you more. We do need to be careful with this.

Steven Brown’s picture

Checking in on an update. Is this something we are willing to commit or just having to wait until the latest version?

Steven Brown’s picture

Steven Brown’s picture

Valeratal’s picture

in alpha-reset.css
table, tbody, tfoot, thead, tr, th, td {
margin: 0;
padding: 0;
border: 0;
outline: 0;
font-size: 100%;
vertical-align: baseline;
font: inherit;
}

"border 0" need remove