id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc	blockedby	blocking	branch_state	votes
3050	Bright colors being used as background colors	egmont	andrew_b	"Please run
{{{
egrep ';(gray|yellow|white|bright)' skins/*.ini
}}}

This lists many occurrences where a bright color, which is only valid as a foreground color in the legacy 16-color world, is being used as background color.

This results in a completely different look with TERM=xterm (16 fg, 8 bg colors) or with TERM=xterm-256color (256 fg+bg colors) settings. With 16-color settings the color reverts to its non-bright counterpart, whereas with 256-color settings the actual bright version is used.

I really doubt that in any of these cases the goal was to be such different in the two cases, or that actually both cases were tested. Probably someone just a picked a color that looked good with their settings, and most likely it doesn't look that good with the other setup.

I believe bright colors shouldn't be used as background colors for this reason. Except, of course, in skins that were explicitly designed for the 256-color world.

(Sibling of ticket #3048.)"	defect	closed	minor	4.8.11	mc-skin	4.8.9	fixed					merged	committed-master