1282280 - Give viewPickerPopup and folderLocationPopup a menu look

Status: RESOLVED FIXED

(Thunderbird :: Theme, defect)

Description

[Richard Marti (:Paenglab)]

From bug 1280429 comment 5:
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 1

[Richard Marti (:Paenglab)]

Attachment: pickerColors.patch

This gives the menupopup[type="folder"], the popup which are showing the message folders, the menu appearance. And I also added rules for the #viewPickerPopup, the popup from Mail Views menulist, to look similar.

Comment 2

[Jorg K (CEST = GMT+2)]

Attachment: Picture showing the folder menu list on the filters panel.

Can you please point out all the places I need to look at and/or attach a/some screenshot(s).

I've looked around the menulist to select folders in the filter definition; I noticed the vertical line, but I think there is padding missing to the right of it.

Comment 3

[Richard Marti (:Paenglab)]

Attachment: pickerColors.patch v2

The only menulist (which I know) I've not checked, and it has menuitems in it. All other have only menu.

Menulists I know:
FolderPicker in main toolbar (needs to add through customize)
Filters for: in Message Filters
Run selected Filter(s) on: in MessageFolter
Choose Folder in Filter Editor
The folder menulists in Copies & Folders of Account Manager (grey menupopup on grey dialog background isn't optimal, but this ones aren't used often)

Maybe some more but I'm not aware of them.

Comment 4

[Jorg K (CEST = GMT+2)]

Comment on attachment 8765233 [details] [diff] [review]
pickerColors.patch v2

Review of attachment 8765233 [details] [diff] [review]:
-----------------------------------------------------------------

I've checked the folder menulist mentioned in comment #3 on Windows 7 and they look good now (and better than without the patch). I'm not really an expert on the CSS stuff and I assume that I may not have covered all the cases. Given that our other reviewers are on Linux or Mac, I guess we'll have to live with my humble review ;-(

Comment 5

[Richard Marti (:Paenglab)]

http://hg.mozilla.org/comm-central/rev/eabc64bac050

Comment 6

[Jorg K (CEST = GMT+2)]

(In reply to Jorg K (GMT+2, PTO during summer, NI me) from comment #4)
> I've checked the folder menulist mentioned in comment #3 on Windows 7 and
> they look good now ...
I've just checked it on Windows 10 and I can't see a vertical line. Is that intentional or am I missing something? Would you like a screenshot?

Comment 7

[Richard Marti (:Paenglab)]

Windows 10 has no vertical lines in menus.

Comment 8

[:aceman]

Maybe the problem is also on Linux :)

So which version was chosen here? Are now all levels of the folder picker coming from a menulist a 3d-relief menu, or are they all white flat?

Comment 9

[Jorg K (CEST = GMT+2)]

(In reply to Richard Marti (:Paenglab) from comment #7)
> Windows 10 has no vertical lines in menus.
Yes, indeed.

Comment 10

[Richard Marti (:Paenglab)]

(In reply to :aceman from comment #8)
> Maybe the problem is also on Linux :)

I've checked this on Linux and it looked good for me. But I can recheck this afternoon.

Comment 11

[Richard Marti (:Paenglab)]

Checked again on Linux. The menulist menupopups and menuitems have the  menupop or menuitem -moz-appearance and are looking correct.

Comment 12

[:aceman]

Yes, looks fine to me on Linux before & after the patch (=> needs no fix).