379985 - [Pinstripe] Use Highlight and highlighttext where applicable

Status: RESOLVED FIXED

(Toolkit :: UI Widgets, defect)

Description

[Stefan [:stefanh]]

Since we now have a real "Higlight" color that is changeable in system prefs, we can get rid of the -moz-mac-alternateprimaryhighlight" stuff. We can also use "highlighttext" instead of a hard-coded white color.

Comment 1

[Stefan [:stefanh]]

Attachment: Don't use -moz-mac-alternateprimaryhighlight and hard-coded white color

Comment 2

[Asaf Romano (gone)]

Comment on attachment 265387 [details] [diff] [review]
Don't use -moz-mac-alternateprimaryhighlight and hard-coded white color

r=mano on the tookit/ part.

Comment 3

[Stefan [:stefanh]]

Comment on attachment 265387 [details] [diff] [review]
Don't use -moz-mac-alternateprimaryhighlight and hard-coded white color

David, can you review the mail part? I'm not sure if this require sr... (but if it does, perhaps you could sr it as well - there's no flag to set).

Comment 4

[David :Bienvenu]

Comment on attachment 265387 [details] [diff] [review]
Don't use -moz-mac-alternateprimaryhighlight and hard-coded white color

looks simple, and fine for the mail part. thx, Stefan

Comment 5

[Stefan [:stefanh]]

Comment on attachment 265387 [details] [diff] [review]
Don't use -moz-mac-alternateprimaryhighlight and hard-coded white color

It appears that r or sr from mcscott/bienvenu is enough to land the mail part.

Comment 6

[Stefan [:stefanh]]

Checking in mail/themes/pinstripe/mail/mailWindow1.css;
/cvsroot/mozilla/mail/themes/pinstripe/mail/mailWindow1.css,v  <--  mailWindow1.css
new revision: 1.11; previous revision: 1.10
done
Checking in toolkit/themes/pinstripe/global/autocomplete.css;
/cvsroot/mozilla/toolkit/themes/pinstripe/global/autocomplete.css,v  <--  autocomplete.css
new revision: 1.7; previous revision: 1.6
done
Checking in toolkit/themes/pinstripe/global/listbox.css;
/cvsroot/mozilla/toolkit/themes/pinstripe/global/listbox.css,v  <--  listbox.css
new revision: 1.6; previous revision: 1.5
done
Checking in toolkit/themes/pinstripe/global/tree.css;
/cvsroot/mozilla/toolkit/themes/pinstripe/global/tree.css,v  <--  tree.css
new revision: 1.15; previous revision: 1.14
done