468857 - <tree class="plain"> doesn't look good on OS X

Status: RESOLVED WONTFIX

(Toolkit :: Themes, defect)

Description

[[e-mail]]

(In reply to bug 395898 comment #10)
> Removing the class "plain" is also good for bug 465402 (otherwise the tree
> header cells wouldn't have any top border on OS X).
We shouldn't go around removing class="plain" from the world just because it's broken on one theme, we should be fixing the theme...

Comment 1

[Markus Stange [:mstange]]

Yep. What's the purpose of class="plain"? Could we just ignore it in Pinstripe?

Comment 2

[[e-mail]]

Well, I believe it was invented to turn off all the margins, borders and padding although unfortunately nobody thought to turn of -moz-appearance so it's slightly broken in winstripe as well.

Comment 3

[[e-mail]]

Attachment: How about:config should look

This with class="plain" style="-moz-appearance: none;" added using DOMi.

Comment 4

[Markus Stange [:mstange]]

Actually, this is how it should look: attachment 349949 [details] ;-)

I haven't really understood this yet. Shouldn't it be the theme's decision how to style a tree in every specific case? Why do we want to put that decision into a class that applies for all platforms?

Comment 5

[Tim Nguyen :ntim]

Is this still relevant ?