305515 - Link icons in Account Central

Status: RESOLVED WORKSFORME

(MailNews Core :: Account Manager, enhancement)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

Build ID: 2005-08-21-05, Windows XP SeaMonkey trunk.

Summary: Link icons in Account Central.

Comment 1

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Patch

Classic doesn't have icons at all, so this is only needed for Modern.

Also, I'm not sure if !important is needed, mscott's pinstripe theme CSS has
it.
I know that !important means that a user can override an author's CSS, but
still not sure if we need this at all.

Comment 2

[[e-mail]]

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

Sorry, I don't like this approach. I think this is best done by rewriting
AccountCentral.xul and I also don't like the way that the hover area for the
text is the height of the image (although the alternative is for hovering the
image not to highlight the text).

Comment 3

[Alfred Kayser]

Please rewrite the XUL to not use a grid/rows, but just groupbox/buttons (so that buttons can have icons).

Comment 4

[:aceman]

It appears to me this is already solved. The icons can be clicked and they are done using CSS background.

Comment 5

[Phoenix]

Resovled per whiteboard