Closed Bug 959983 Opened 8 years ago Closed 7 years ago

NFC: There should be an icon for NFC on the status bar


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

Gonk (Firefox OS)
Not set


(blocking-b2g:2.0+, b2g-v2.0 fixed)

2.0 S1 (9may)
blocking-b2g 2.0+
Tracking Status
b2g-v2.0 --- fixed


(Reporter: allstars.chh, Assigned: gasolin)



(Whiteboard: [FT:System-Platform][p=1])


(1 file)

Like Bluetooth, when we enable NFC there should be an icon for telling user NFC has been enabled.
blocking-b2g: --- → 1.4?
Hi Ken
I think it's a very basic feature for NFC. It should be 1.4+
Flags: needinfo?(kchang)
I wonder if UX team can design this for NFC. By the way, there isn't a NFC status bar in other devices, you could double check it again. Partner also request this, so please talk with them. But I wonder if it is a vendor specific feature.
Flags: needinfo?(kchang) → needinfo?(jhuang)
We've already updated this icon months ago,
Please check
Flags: needinfo?(jhuang)
blocking-b2g: 1.4? → backlog
request to 1.5+
blocking-b2g: backlog → 1.5?
Wesley, who can help this in Gaia.
Flags: needinfo?(whuang)
Candice, is this in systemfe's plate?
Flags: needinfo?(cserran)
No longer blocks: b2g-NFC-2.0
(In reply to Wesley Huang [:wesley_huang] from comment #6)
> Candice, is this in systemfe's plate?

Yes, confirmed this is systemsfe bug. Updating whiteboard tag and added to our backlog.
Flags: needinfo?(cserran)
Whiteboard: [systemsfe]
I'm moving this to backlog for now.  I fully acknowledge that it is quite important for NFC to inform the user of status, but it is not clear that we'll block 1.5 on NFC at this point.  If that changes, we can definitely bump it up.
blocking-b2g: 1.5? → backlog
Flags: needinfo?(whuang)
Wesley, I think NFC is one of key features in 1.5, right?
Flags: needinfo?(whuang)
Yes. NFC (sharing) is key focus in 1.5, so there will be obvious user interactions.
Suggest to implement this icon per defined by UX.
Flags: needinfo?(whuang)
Just realized this status icon is needed for partner in 1.5 timeframe.(bug 972778)
Set to 1.5? again.
blocking-b2g: backlog → 1.5?
For triage information - this is requested by partners as mentioned in comment 11 and they are unable to take this one.
NFC p2p sharing is defined as key focus in 2.0, so I'm setting this + for this is obvious user experience flaw.
blocking-b2g: 2.0? → 2.0+
Duplicate of this bug: 996507
Component: NFC → Gaia::System
Assignee: nobody → gasolin
@greg I mapped NFC stat 0(NFC_HW_STATE_OFF)-> hide icon , 1~3-> show icon on statusbar.
Attachment #8411552 - Flags: feedback?(gweng)
Should ask Yoshi that if `NFC_HW_STATE_DISABLE_DISCOVERY` means it's active.
Flags: needinfo?(allstars.chh)
(In reply to Greg Weng [:snowmantw][:gweng][:λ] from comment #16)
> Should ask Yoshi that if `NFC_HW_STATE_DISABLE_DISCOVERY` means it's active.

It means in low power mode, it cannot detect nor get detected,
but the hardware is still enabled.
Flags: needinfo?(allstars.chh)
So it should be treated as inactive?
When in low power mode, it still could be in card emulation mode, which is related to payment.
But we don't support it for now.

We could file a new bug to discuss this special case.
So it sounds like `stat 0(NFC_HW_STATE_OFF)-> hide icon` is right in current stage.

Do you know how other platform the NFC status is displayed in low power mode?
Flags: needinfo?(allstars.chh)
Asked Juwei (UX): it's OK to show the icon at the `NFC_HW_STATE_DISABLE_DISCOVERY` state.

So Comment 15 is a correct way to implement this.
LG Nexus 5 doesn't have icon for NFC.
Sony still shows NFC icon in low power mode.

Sorry I don't know about Samsung.
Flags: needinfo?(allstars.chh)
Whiteboard: [systemsfe]
Whiteboard: [FT:System-Platform]
Target Milestone: --- → 2.0 S1 (9may)
Whiteboard: [FT:System-Platform] → [FT:System-Platform][p=1]
Comment on attachment 8411552 [details] [review]
pull request redirect to github

travis passed
Attachment #8411552 - Flags: review?(alive)
Comment on attachment 8411552 [details] [review]
pull request redirect to github

r+ if event name is changed and unit test for nfcmanager part
Attachment #8411552 - Flags: review?(alive) → review+
add nfc_manager test case and merged to gaia-master

Closed: 7 years ago
Resolution: --- → FIXED
Blocks: 1005797
Attachment #8411552 - Flags: feedback?(gweng)
Blocks: 1023765
You need to log in before you can comment on or make changes to this bug.