1280429 - Filter definition uses menulists with separators which are too short

Status: RESOLVED FIXED

(Thunderbird :: Theme, defect)

Description

[Jorg K (CEST = GMT+2)]

Filter definition uses menulists with separators which are too short:

Tools > Message Filters, New.

Below under "Perform these actions": Both menulists have separators which are too short.

Comment 1

[Richard Marti (:Paenglab)]

Jörg, the choose folder menulist, the second, is correct. It has icons, but I confirm the first needs fix.

Comment 2

[Jorg K (CEST = GMT+2)]

Attachment: Screenshot showing that there is no vertical line.

Sure, the short separator is OK for the second one, but then the vertical line is missing. Compare this to, for example, "move message".

Comment 3

[Richard Marti (:Paenglab)]

Attachment: Bug1280429.patch

This fixes the the separator lehght and I found on Linux some menupopups like Tags, Priority and Junk had not needed indentation.

This patch needs bug 1278489 applied first.

Comment 4

[Richard Marti (:Paenglab)]

(In reply to Jorg K (GMT+2, PTO during summer, NI me) from comment #2)
> Created attachment 8763081 [details]
> Screenshot showing that there is no vertical line.
> 
> Sure, the short separator is OK for the second one, but then the vertical
> line is missing. Compare this to, for example, "move message".

The vertical line is from native appearance menus. The menupopups are not native. They are white for example and the menus gray.

Comment 5

[:aceman]

(In reply to Richard Marti (:Paenglab) from comment #4)
> > Sure, the short separator is OK for the second one, but then the vertical
> > line is missing. Compare this to, for example, "move message".
> 
> The vertical line is from native appearance menus. The menupopups are not
> native. They are white for example and the menus gray.

Yes, but it is a mess in the filter editor, that the first menupopup of the picker is white and flat but the subsequent menupoups coming from menus are gray and 3d relief (and possibly with the vertical lines). I vaguely remember we wanted to fix this somewhere but can't remember if it was successful. That should be discussed in a new bug as it is not straightforward (there are multiple options to do).

Comment 6

[:aceman]

Comment on attachment 8763331 [details] [diff] [review]
Bug1280429.patch

I'll try this out, but in the mean time we need to decide what to do with Semankey as I think you do not add the classes to SM (in the preceding bug).

I'm not even sure SM needs these new classes as their themes may work differently.

In worst case adding the calls to the classes here (mailnews file) will have no effect in SM, but fixes TB.

Comment 7

[Richard Marti (:Paenglab)]

Yes, the background color should be done in a new bug.

This additional classes don't harm SM as they have no rule for this class. For me, the worst case would be, it fixes TB and messes up SM. But as I wrote before, this classes aren't used by SM. If they want, they can add this rules to fix their themes if needed.

Comment 8

[Richard Marti (:Paenglab)]

Bug 1278489 fixes also this issue.

Comment 9

[Jorg K (CEST = GMT+2)]

Comment on attachment 8763331 [details] [diff] [review]
Bug1280429.patch

This is obsolete, right, and no review required?

Comment 10

[Richard Marti (:Paenglab)]

Ah yes, sorry.