133197 - alert.css in Classic theme should use system colors

Status: RESOLVED WORKSFORME

(Toolkit :: Themes, defect)

Description

[John Dobbins]

mozilla/themes/classic/communicator/alerts/alert.css uses the same colors as the
Modern theme instead of using system colors.

Comment 1

[John Dobbins]

Attachment: system colors

added patch

Comment 2

[Boris Zbarsky [:bzbarsky]]

Are those system colors correct for the Mac? Or do we need a separate alert.css
there?

Comment 3

[John Dobbins]

I Don't have a Mac. None of the other files outside of the global directory show
any differance between Mac and Windows, so it is as correct for Macs as any
other file in the Communicator directory.

Comment 4

[Boris Zbarsky [:bzbarsky]]

ccing mpt, who has often expressed his views on the color issues these sorts of
things have on the Mac....

Comment 5

[Stefan [:stefanh]]

Not sure if this is relevant anymore. Nearest match nowadays is toolkit/themes/winstripe/global/alerts/alert.css which doesn't use system colors. The gnomestripe version does, though.