616699 - Address Book Output Panel does not read (or overrides) GTK theme settings for background, text color.

Status: RESOLVED FIXED

(SeaMonkey :: UI Design, defect)

Description

[skyalmian]

User-Agent:       Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.17pre) Gecko/20101201 SeaMonkey/2.0.12pre
Build Identifier: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.17pre) Gecko/20101201 SeaMonkey/2.0.12pre

While the Address Book's list of contacts and address book panels (top row and left column) adhere to whatever setting the GTK theme is set to, namely text and background color, the output panel below overrides with white background and white text. This is only noticeable on dark GTK themes with black backgrounds and white or gray text.

Reproducible: Always

Steps to Reproduce:
1. Switch to a dark-themed GTK theme where black backgrounds are the norm.



A screenshot for visual example:
http://sky.sploitz.com/bugzilla/seamonkey-address-book.png

Comment 1

[rsx11m]

This looks like the SeaMonkey equivalent of Thunderbird bug 351736?
Address book backend is shared code, but the themes aren't...

Comment 2

[skyalmian]

Yep... I incorporated the changes into addressbook.css from "Comment on attachment 375176 [details] [diff] [review]" and tested... Like Brian said: "looks good", with and without dark GTK theme. "The fix v2" needs to be incorporated into addressbook.css.

Comment 3

[rsx11m]

Since you've worked it out already, do you like to post a patch for review? :-D

Comment 4

[skyalmian]

Attachment: Patch

I've never done any actual coding before and this is first time doing anything... Does this count?

Comment 5

[rsx11m]

Comment on attachment 495309 [details] [diff] [review]
Patch

Looks good to me, I'm asking Neil for review on your behalf.

Comment 6

[[e-mail]]

Comment on attachment 495309 [details] [diff] [review]
Patch

> .CardViewLink {
>-  color: blue;
>-  text-decoration: underline;
>+  color: -moz-nativehyperlinktext;
Why are you removing the underlining?

Comment 7

[rsx11m]

The Thunderbird themes have "text-decoration: underline;" defined later as part of the "html|a" rules, SeaMonkey's classic theme doesn't have that.

Comment 8

[[e-mail]]

Comment on attachment 495309 [details] [diff] [review]
Patch

> .CardViewLink {
>-  color: blue;
>-  text-decoration: underline;
>+  color: -moz-nativehyperlinktext;
r=me if you restore the underlining.

Comment 9

[skyalmian]

Attachment: "hg diff -p -U 8 addressbook.css > addressbook2.txt" output. r=Neil [Checkin: comment 14]

Re-added the line...

Comment 10

[rsx11m]

This should be approved after Neil's r+, unless I'm reading anything wrong.
Thus, I'm setting the "checkin-needed" keyword for someone to push on trunk.

Comment 11

[skyalmian]

Thank you for everything you've done... This wouldn't have been solved had you not found the similar Thunderbird bug. :} I did look before posting this, but I had limited my searches to SeaMonkey...

Comment 12

[rsx11m]

You are welcome, and congratulations for your first SeaMonkey patch!

Comment 13

[rsx11m]

I'm not aware of any tree closures on trunk, thus a friendly ping for a push?

Comment 14

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 495413 [details] [diff] [review]
"hg diff -p -U 8 addressbook.css > addressbook2.txt" output. r=Neil [Checkin: comment 14]

http://hg.mozilla.org/comm-central/rev/1e36326c45e3

Comment 15

[Jens Hatlak (:InvisibleSmiley)]

Thanks!

Comment 16

[[e-mail]]

Comment on attachment 495413 [details] [diff] [review]
"hg diff -p -U 8 addressbook.css > addressbook2.txt" output. r=Neil [Checkin: comment 14]

> .CardViewLink {
>-  color: blue;
>   text-decoration: underline;
>+  color: -moz-nativehyperlinktext;
[The slip over the underline has caused the change in colour to migrate!]