Last Comment Bug 808974 - Make the "Recent" submenu of folder picker also inherit the class of the parent menu.
: Make the "Recent" submenu of folder picker also inherit the class of the pare...
: polish
Product: MailNews Core
Classification: Components
Component: Filters (show other bugs)
: Trunk
: All All
-- normal (vote)
: Thunderbird 20.0
Assigned To: :aceman
Depends on: 315367
Blocks: 809066 814041
  Show dependency treegraph
Reported: 2012-11-06 01:39 PST by :aceman
Modified: 2012-11-21 09:07 PST (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

screenshot of how the Recent folders has different style (12.91 KB, image/png)
2012-11-06 01:39 PST, :aceman
no flags Details
screenshot of how the normal subfolders have the same style as the parent (11.44 KB, image/png)
2012-11-06 01:39 PST, :aceman
no flags Details
patch (1.26 KB, patch)
2012-11-06 11:26 PST, :aceman
neil: review+
bwinton: ui‑review-
Details | Diff | Splinter Review
patch 2 (alternative) (2.99 KB, patch)
2012-11-12 10:48 PST, :aceman
bwinton: review+
bwinton: ui‑review+
Details | Diff | Splinter Review

Description User image :aceman 2012-11-06 01:39:09 PST
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.
Comment 1 User image :aceman 2012-11-06 01:39:59 PST
Created attachment 678660 [details]
screenshot of how the normal subfolders have the same style as the parent
Comment 2 User image :aceman 2012-11-06 11:26:37 PST
Created attachment 678823 [details] [diff] [review]

Should be tested on Win/Mac, on Linux I see no difference (all the pickers are grey, on Win some are white).
Comment 3 User image Blake Winton (:bwinton) (:☕️) 2012-11-12 09:18:49 PST
Comment on attachment 678823 [details] [diff] [review]

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

Check out the screenshot at

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.  ;)

Comment 4 User image :aceman 2012-11-12 10:38:11 PST
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.
Comment 5 User image Blake Winton (:bwinton) (:☕️) 2012-11-12 10:45:10 PST
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.
Comment 6 User image :aceman 2012-11-12 10:48:47 PST
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.
Comment 7 User image Blake Winton (:bwinton) (:☕️) 2012-11-12 10:50:43 PST
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 User image 2012-11-12 11:16:36 PST
Indeed, the menulist-menupopup class is missing from Thunderbird's version of the filter list dialog.
Comment 9 User image :aceman 2012-11-12 11:18:34 PST
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 10 User image Blake Winton (:bwinton) (:☕️) 2012-11-19 10:05:36 PST
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.
Comment 11 User image :aceman 2012-11-19 11:42:43 PST
Comment on attachment 680701 [details] [diff] [review]
patch 2 (alternative)

OK, so let's continue with this version.
Comment 12 User image :aceman 2012-11-20 08:14:30 PST
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.
Comment 13 User image Ryan VanderMeulen [:RyanVM] 2012-11-20 17:05:32 PST

Note You need to log in before you can comment on or make changes to this bug.