356865 - Remove hard-coded color:black and use -moz-activehyperlinktext for links

Status: RESOLVED FIXED

(SeaMonkey :: Help Documentation, defect)

Description

[Stefan [:stefanh]]

I recall that there where some discussion about this when toolkit fixed their css... Finally remembered it :)

Comment 1

[Stefan [:stefanh]]

Attachment: Fix up the css

I also simplified some style rules and removed one unused class.

Comment 2

[[e-mail]]

Comment on attachment 242427 [details] [diff] [review]
Fix up the css

Why the 100% width?

Comment 3

[[e-mail]]

(In reply to comment #2)
>Why the 100% width?
Whoops, you didn't actually change that line ;-)

Comment 4

[[e-mail]]

So, what was the issue? We're already hardcoding a bunch of colours. I'd prefer hardcode a white background and be done with it, except that won't work with KaiRo's grand scheme of things whereby he wants to use toolkit's help on help.

Comment 5

[Stefan [:stefanh]]

(In reply to comment #4)
> So, what was the issue? We're already hardcoding a bunch of colours. I'd prefer
> hardcode a white background and be done with it, except that won't work with
> KaiRo's grand scheme of things whereby he wants to use toolkit's help on help.

See bug 274471. I recall that you pinged me on irc about it. But that was a long time ago of course :-)

Comment 6

[Stefan [:stefanh]]

Comment on attachment 242427 [details] [diff] [review]
Fix up the css

This is just some simple css cleanup in help.

Comment 7

[Robert Kaiser]

Comment on attachment 242427 [details] [diff] [review]
Fix up the css

a=me for 1.1b

Comment 8

[Stefan [:stefanh]]

Attachment: Branch diff

Here's a branch diff. The trunk patch doesn't apply on the 1.8 version of the file.

Comment 9

[Stefan [:stefanh]]

Checked in by Mark B (trunk/branch).