Closed Bug 885677 Opened 6 years ago Closed 6 years ago

No moz-icon when default sound is selected

Categories

(Calendar :: Preferences, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Paenglab, Assigned: Paenglab)

Details

Attachments

(1 file)

In Options -> Reminders when the sound is on default (chrome://calendar/content/sound.wav) where is no moz-icon shown.

This comes from the if/else where the icon is only set when the file is a "file://) URL.
Attached patch patchSplinter Review
Move the moz-icon assignment after the if/else to always be done. I've only tested it on Win8 and it works here. It should work also on the other platforms.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #765861 - Flags: review?(philipp)
Attachment #765861 - Flags: review?(philipp) → review+
Checked in to comm-central: http://hg.mozilla.org/comm-central/rev/1d81563ab6cd
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.6
You need to log in before you can comment on or make changes to this bug.