Closed
Bug 808974
Opened 12 years ago
Closed 12 years ago
Make the "Recent" submenu of folder picker also inherit the class of the parent menu.
Categories
(MailNews Core :: Filters, defect)
MailNews Core
Filters
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 20.0
People
(Reporter: aceman, Assigned: aceman)
References
Details
(Keywords: polish)
Attachments
(3 files, 1 obsolete file)
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
patch
|
bwinton
:
review+
bwinton
:
ui-review+
|
Details | Diff | Splinter Review |
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.
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)
Updated•12 years ago
|
Attachment #678823 -
Flags: review?(neil) → review+
Comment 3•12 years ago
|
||
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-
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)
Comment 5•12 years ago
|
||
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)
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)
Comment 7•12 years ago
|
||
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•12 years ago
|
||
Indeed, the menulist-menupopup class is missing from Thunderbird's version of the filter list dialog.
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•12 years ago
|
||
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•12 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)
Attachment #678823 -
Attachment is obsolete: true
Assignee | ||
Comment 12•12 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)
Keywords: checkin-needed
Comment 13•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 20.0
You need to log in
before you can comment on or make changes to this bug.
Description
•