Closed Bug 1073485 Opened 10 years ago Closed 10 years ago

[Ringtones]In new notification, ringtone icon is not there

Categories

(Firefox OS Graveyard :: Gaia::Ringtones, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: ankit93040, Unassigned)

Details

(Whiteboard: [g+][LibGLA,TD101195,QE2, B] )

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/37.0.2062.124 Safari/537.36 Steps to reproduce: 3. Tester`s Action : Music - select any music file - Share - Ringtones - Save - reserve Notification message Actual results: only text message seen Expected results: rigntone icon with message should come
Whiteboard: [g+][LibGLA,TD101195,QE2, B]
Attached patch 1073485.patchSplinter Review
Patch for MOz v2.0
Attachment #8495903 - Flags: review?(squibblyflabbetydoo)
Comment on attachment 8495903 [details] [diff] [review] 1073485.patch You need UI review for this first, but I don't think it makes sense to show the icon given that this is a transient notification (and we'll eventually be styling transient notifications even more differently).
Attachment #8495903 - Flags: review?(squibblyflabbetydoo) → ui-review?(kcaldwell)
(In reply to Jim Porter (:squib) from comment #2) > You need UI review for this first, but I don't think it makes sense to show > the icon given that this is a transient notification (and we'll eventually > be styling transient notifications even more differently). Correct, confirmation notifications that do not require any additional user action, do not display an icon.
Comment on attachment 8495903 [details] [diff] [review] 1073485.patch -ui as this isn't a bug, the notification does not require an icon.
Attachment #8495903 - Flags: ui-review?(kcaldwell) → ui-review-
Resolving this as its not a bug. Thank you
Status: UNCONFIRMED → RESOLVED
Closed: 10 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: