Page MenuHomePhabricator

remove table { color: black } in Vector and Monobook
Closed, ResolvedPublic

Description

table { color: black } in
phase3/skins/monobook/main.css and
phase3/skins/vector/screen.css
is not neccessary, because Vector and Monobook defines
div#content { color: black; }

Bug 26708 removed
table { background-color: white }
so the default background color for tables is transparent.

At the moment tables in dark background color container are inreadable:
<div style="background-color:DarkBlue; color:white">
Table
{|

left
right
}

</div>

Maybe wikitable should get
table.wikitable { color: black }
because wikitable already has
table.wikitable { background-color: #f9f9f9 }


Version: 1.18.x
Severity: minor

Details

Reference
bz28422

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:33 PM
bzimport set Reference to bz28422.

Tables have inheritance turned off by default in some browsers.

(In reply to comment #1)

Tables have inheritance turned off by default in some browsers.

Skin Modern has also no definition for the text color for tables.

(In reply to comment #0)

table { color: black } in
phase3/skins/monobook/main.css and
phase3/skins/vector/screen.css
is not neccessary, because Vector and Monobook defines
div#content { color: black; }

Indeed.

At the moment tables in dark background color container are inreadable:

Very true.

Maybe wikitable should get table.wikitable { color: black }
because wikitable already has table.wikitable { background-color: #f9f9f9 }

.wikitable nees to keep color:black because of it's backgound color, but since tables in general no longer have a background color (bug 26708) they shouldn't have a text color anymore.

Will remove black from table and add to wikitable.