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 bytesnilesh.addweb

Comments

zany’s picture

zany commented

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

zany commented
imshivani’s picture

imshivani commented
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.

nilesh.addweb’s picture

nilesh.addweb commented
StatusFileSize
new invalid_css-1834376-4.patch327 bytes

PFA patch.

nilesh.addweb’s picture

nilesh.addweb commented
Status: Active » Needs review
zany’s picture

zany commented
Status: Needs review » Reviewed & tested by the community

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

giupenni’s picture

giupenni commented

+1

kunal_kapoor’s picture

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

kunal_kapoor commented
Status: Fixed » Closed (fixed)
kunal_kapoor’s picture

kunal_kapoor commented