adjust the appearance of mailviews picker in the Thunderbird toolbar

VERIFIED FIXED in Thunderbird 25.0

Status

Thunderbird
Theme
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: aceman, Assigned: Paenglab)

Tracking

({polish})

Trunk
Thunderbird 25.0
All
Windows XP
polish

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments, 1 obsolete attachment)

(Reporter)

Description

5 years ago
Created attachment 755335 [details]
screenshot with marked height difference

Bug 861933 unified the height of the mailviews picker with the folder location picker in the Thunderbird toolbar in Windows 7. This should be done for other OSes as needed. Also, the "Tags" and "Custom views" are indented in the list. This was changed for Win7 but not other OSes. Screenshot from Win XP attached.
(Assignee)

Comment 1

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

This patch needs bug 876222 applied first.

OS X looks good.

On Linux I gave in locationFolders menulist the items in subfolders 3px padding on the left to give them the same gap to the edges as the main menu in this menulist have. Aceman, please can you check this if it also looks good on your Linux distribution?

On WinXP I used some rules from mailWindow1-aero.css and removed them there.
 - fixed is the height two menulists
 - the indented menus are also fixed
 - the items left and right gap to the menu edges are now all 5px
 - the sub-menuitems in viewPicker are 2px taller to not look so condensed
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #756114 - Flags: ui-review?(mconley)
Attachment #756114 - Flags: review?(mconley)
(Reporter)

Comment 2

5 years ago
I am not sure I see any difference. Where should I look?
(Assignee)

Comment 3

5 years ago
In locationsFolders all menu/menuitems in all menus and submenus should have the same gap between the text and the border on the left.
(Reporter)

Comment 4

5 years ago
Created attachment 757517 [details]
screenshot on Linux with the patch

I still do not see what the gain is. The padding on the left is now much bigger than without the patch. Is that positive?
(Assignee)

Comment 5

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

On Linux the first level has also the id folderLocationPopup. I added menu > to the selectors to apply only beginning with the second folderLocationPopup. Now all items have the same padding to the icons of 5px.
Attachment #756114 - Attachment is obsolete: true
Attachment #756114 - Flags: ui-review?(mconley)
Attachment #756114 - Flags: review?(mconley)
Attachment #757568 - Flags: ui-review?(mconley)
Attachment #757568 - Flags: review?(mconley)
Comment on attachment 757568 [details] [diff] [review]
patch

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

Thanks Richard!
Attachment #757568 - Flags: ui-review?(mconley)
Attachment #757568 - Flags: ui-review+
Attachment #757568 - Flags: review?(mconley)
Attachment #757568 - Flags: review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/6cdc08bdc5fc
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 25.0
(Reporter)

Comment 8

5 years ago
Looks great on XP, thank.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.