Closed Bug 945773 Opened 11 years ago Closed 11 years ago

[Settings] 1.3 Visual Refresh - Add NFC Icon

Categories

(Firefox OS Graveyard :: Gaia::Settings, defect, P1)

x86
Gonk (Firefox OS)
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: amylee, Assigned: pivanov)

References

Details

(Whiteboard: visual design, visual-tracking, jian)

Attachments

(4 files)

Hi Pavel,

I the NFC header is using the wrong icon (airplane). I've added the NFC icon to the sprite sheets. Can you please update? 

Thanks!
Assignee: amylee.design → pivanov
Attached image settings_mock.gif
Hi Pavel, 

Here is a mock of the settings screen with where the icon needs to be replaced. The right side shows the correct icon used for NFC. 

Thanks!
Attached image After patch screenshot
Attachment #8341902 - Flags: feedback?(amylee.design)
(In reply to Pavel Ivanov [:ivanovpavel] from comment #2)
> Created attachment 8341902 [details]
> After patch screenshot

This looks good. Thanks Pavel!
Attachment #8341902 - Flags: feedback?(amylee.design) → feedback+
Attached file patch for Gaia/master
Attachment #8341922 - Flags: review?(epang)
Comment on attachment 8341922 [details] [review]
patch for Gaia/master

Thanks Pavel, looks good!  We can land it :)
Attachment #8341922 - Flags: review?(epang) → review+
Landed to master:
https://github.com/mozilla-b2g/gaia/commit/2adbf9c9c19b59dfa40d8871e22f32765cc34c39
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: