The CSS property 'text-align' does not allow for a value of 'auto'. See http://www.w3.org/TR/css3-text/#text-align0
Linie 96 in TAO's drupal.css reads:
thead th { text-align:auto; }
http://drupalcode.org/project/tao.git/blob/refs/heads/7.x-3.x:/drupal.cs...
Maybe this is some kind of hack? It is ignored in tested browsers though.
Thanks!

CommentFileSizeAuthor
#4 invalid_css-1834376-4.patch327 bytesankur.addweb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

zany’s picture

Follow up: I tested the behavior and center or inherit is the value we want here. Since the current value is invalid this is what we actually get right now.

zany’s picture

imshivani’s picture

Issue summary: View changes

Yes, text-align: auto isn't the right way to get the thing done.
it should be thead th { text-align:center; }
instead of
thead th { text-align:auto; } in the line number 95.

ankur.addweb’s picture

FileSize
327 bytes

PFA patch.

ankur.addweb’s picture

Status: Active » Needs review
zany’s picture

Status: Needs review » Reviewed & tested by the community

RTBC. The CSS is valid and the visual apperance unchanged.

giupenni’s picture

+1

kunalkapoor’s picture

Status: Reviewed & tested by the community » Fixed
kunalkapoor’s picture

Status: Fixed » Closed (fixed)
kunalkapoor’s picture