Remove unneeded menuitem-non-iconic class in calendar-alarm-widget.xml

RESOLVED FIXED in 6.2

Status

enhancement
RESOLVED FIXED
Last year
Last year

People

(Reporter: Paenglab, Assigned: Paenglab)

Tracking

unspecified

Details

Attachments

(1 attachment)

Assignee

Description

Last year
Bug 1440146 removes the menuitem-non-iconic binding. Calendar uses this in one place.
Assignee

Comment 1

Last year
I tried it without this class and saw no difference.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8956184 - Flags: review?(philipp)
Comment on attachment 8956184 [details] [diff] [review]
menuitem-non-iconic.patch

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

This class used to leave an empty space if the menu contained both items with and without icons. I think it was removed in Firefox though so lets make it go away here as well.
Attachment #8956184 - Flags: review?(philipp) → review+
Assignee

Comment 3

Last year
Thanks.
Keywords: checkin-needed

Comment 4

Last year
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/a1566833de20
Remove unneeded menuitem-non-iconic class in calendar-alarm-widget.xml. r=philipp
Status: ASSIGNED → RESOLVED
Closed: Last year
Keywords: checkin-needed
Resolution: --- → FIXED

Updated

Last year
Target Milestone: --- → 6.2
You need to log in before you can comment on or make changes to this bug.