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)
Firefox OS Graveyard
Gaia::Ringtones
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: ankit93040, Unassigned)
Details
(Whiteboard: [g+][LibGLA,TD101195,QE2, B] )
Attachments
(1 file)
2.00 KB,
patch
|
kcaldwell
:
ui-review-
|
Details | Diff | Splinter Review |
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
Reporter | ||
Updated•10 years ago
|
Whiteboard: [g+][LibGLA,TD101195,QE2, B]
Reporter | ||
Comment 1•10 years ago
|
||
Patch for MOz v2.0
Attachment #8495903 -
Flags: review?(squibblyflabbetydoo)
Comment 2•10 years ago
|
||
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-
Reporter | ||
Comment 5•10 years ago
|
||
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.
Description
•