The default bug view has changed. See this FAQ.

Make the "Recent" submenu of folder picker also inherit the class of the parent menu.

RESOLVED FIXED in Thunderbird 20.0

Status

MailNews Core
Filters
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: aceman, Assigned: aceman)

Tracking

(Blocks: 1 bug, {polish})

Trunk
Thunderbird 20.0
polish
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments, 1 obsolete attachment)

(Assignee)

Description

5 years ago
Created attachment 678659 [details]
screenshot of how the Recent folders has different style

Found in bug 315367 comment 51.

The recent submenu of file picker widget may have a different style to the parent folder. The submenus (folders) of normal accounts do inherit the parent menu's style, but Recent does not. See the attached screenshots.
(Assignee)

Comment 1

5 years ago
Created attachment 678660 [details]
screenshot of how the normal subfolders have the same style as the parent
(Assignee)

Comment 2

5 years ago
Created attachment 678823 [details] [diff] [review]
patch

Should be tested on Win/Mac, on Linux I see no difference (all the pickers are grey, on Win some are white).
Attachment #678823 - Flags: ui-review?(bwinton)
Attachment #678823 - Flags: review?(neil)
(Assignee)

Updated

5 years ago
Status: NEW → ASSIGNED
(Assignee)

Updated

5 years ago
Blocks: 809066

Updated

4 years ago
Attachment #678823 - Flags: review?(neil) → review+
Comment on attachment 678823 [details] [diff] [review]
patch

So, on Mac, I'm not a fan of this change.

Check out the screenshot at https://dl.dropbox.com/u/2301433/Screenshots/SmallFolders.png

I think the sub-menus should be in the bigger font.  Now, perhaps we should just set the class, and do this in the CSS, but I would like to see this fixed before I give it a ui-r+.  (Feel free to ping Paenglab, if you don't feel like mucking around in the CSS yourself.  ;)

Thanks,
Blake.
Attachment #678823 - Flags: ui-review?(bwinton) → ui-review-
(Assignee)

Comment 4

4 years ago
Hey, this patch only affects the Recent menu. You can't lookup random bugs in the style of the submenu of the account items :) Or I don't understand what you see. Please show me all those 3 submenus before the patch and after the patch (so 6 images), not a mixture of each place showing with using different code (before or after patch) ;) Also you must compare the submenus inside one picker (this bug should make the font the same in Recent submenu and the account submenus), not between different dialogs. Each of them may be setting a different class of it.
Flags: needinfo?(bwinton)
Sure, but I think all the other menus should act like the Recent menu currently does, not the other way around, which means removing the class from the other menus, not adding the class to the Recent menu.
Flags: needinfo?(bwinton)
(Assignee)

Comment 6

4 years ago
Created attachment 680701 [details] [diff] [review]
patch 2 (alternative)

But if you want to get all pickers in all dialogs to have the same style then I can add class="menulist-menupopup" where it is missing (e.g. in the filter list "run filters on" picker). You can try with this patch both of the pickers in the filter list dialog and the one in the filter editor. But test it on all platforms.
Attachment #680701 - Flags: review?(bwinton)
Trying it on Linux is a giant pain for me, could I get you to post screenshots of that platform, and I'll try it on Windows and Mac?

Comment 8

4 years ago
Indeed, the menulist-menupopup class is missing from Thunderbird's version of the filter list dialog.
(Assignee)

Comment 9

4 years ago
None of the patches changes anything on Linux so no screenshot needed :) It seems the default style is a heavy grey menu with 3d border and that is what is used on all pickers. Also the menulist-menupopup class looks the same. Only Windows and Mac have this thin white menu. But the Recent submenu was grey on Windows before the patch. If the actual style specified in the class is not what you want then that is a different problem, we can do it but first need to get all the pickers to use the same class (patch 2 is a start for the filters dialog).

Thanks for confirmation Neil. I've added them in patch 2.
Comment on attachment 680701 [details] [diff] [review]
patch 2 (alternative)

Looks good on Mac, looks good on Windows.  There's another bug, but I'll file that separately and cc you.  In the meantime, r=me.
Attachment #680701 - Flags: ui-review+
Attachment #680701 - Flags: review?(bwinton)
Attachment #680701 - Flags: review+
(Assignee)

Comment 11

4 years ago
Comment on attachment 680701 [details] [diff] [review]
patch 2 (alternative)

OK, so let's continue with this version.
Attachment #680701 - Flags: review?(mkmelin+mozilla)
(Assignee)

Updated

4 years ago
Attachment #678823 - Attachment is obsolete: true
(Assignee)

Comment 12

4 years ago
Comment on attachment 680701 [details] [diff] [review]
patch 2 (alternative)

Oh, bwinton gave r+ too :) And for the code change we have r+ from Neil.
Attachment #680701 - Flags: review?(mkmelin+mozilla)
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/b8be8ffe9f7a
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 20.0
(Assignee)

Updated

4 years ago
Blocks: 814041
You need to log in before you can comment on or make changes to this bug.