Closed Bug 1035237 Opened 6 years ago Closed 6 years ago

The labels in the menulist items for the mail view selector don't line up vertically

Categories

(SeaMonkey :: MailNews: General, defect)

x86_64
Windows 7
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.30

People

(Reporter: philip.chee, Assigned: philip.chee)

References

Details

Attachments

(1 file)

This was noticed while working on bug 1023277

STR: On windows 7 using an aero theme.
1. Open the main mailnews window. Select a folder with messages in it.
2. Click or Select the Mail View Selector menulist to open it.
3. Notice that the labels don't all line up vertically. Actually they are all over the place.
This patch adds the menu-iconic or menuitem-iconic class to the menus and menuitems in the menulist.
Assignee: nobody → philip.chee
Status: NEW → ASSIGNED
Attachment #8451679 - Flags: feedback?(neil)
Attachment #8451679 - Flags: feedback?(neil) → feedback+
Comment on attachment 8451679 [details] [diff] [review]
Patch v1.0 Add *-iconic classes to the menu/menuitems.

> This was noticed while working on bug 1023277
> 
> STR: On windows 7 using an aero theme.
> 1. Open the main mailnews window. Select a folder with messages in it.
> 2. Click or Select the Mail View Selector menulist to open it.
> 3. Notice that the labels don't all line up vertically. Actually they are
> all over the place.
Attachment #8451679 - Flags: superreview?(mnyromyr)
Attachment #8451679 - Flags: review?(iann_bugzilla)
Comment on attachment 8451679 [details] [diff] [review]
Patch v1.0 Add *-iconic classes to the menu/menuitems.

/sigh/ moving review request to Neil - Mnyromyr said that a review by Neil can be taken as a moa+
Attachment #8451679 - Flags: superreview?(mnyromyr)
Attachment #8451679 - Flags: review?(neil)
Attachment #8451679 - Flags: review?(iann_bugzilla)
Attachment #8451679 - Flags: review?(neil) → review+
Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/a31848b11c6e
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.30
You need to log in before you can comment on or make changes to this bug.