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

RESOLVED FIXED in Firefox OS v2.0

Status

Firefox OS
Gaia::System
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: allstars, Assigned: gasolin)

Tracking

(Blocks: 3 bugs)

unspecified
2.0 S1 (9may)
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

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

Details

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

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
Like Bluetooth, when we enable NFC there should be an icon for telling user NFC has been enabled.
(Reporter)

Updated

4 years ago
blocking-b2g: --- → 1.4?
(Reporter)

Updated

4 years ago
Blocks: 949293
(Reporter)

Updated

4 years ago
Blocks: 972778
(Reporter)

Comment 1

4 years ago
Hi Ken
I think it's a very basic feature for NFC. It should be 1.4+
Flags: needinfo?(kchang)

Comment 2

4 years ago
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)

Comment 3

4 years ago
We've already updated this icon months ago,
Please check https://github.com/mozilla-b2g/gaia/tree/master/apps/system/style/statusbar/images
Flags: needinfo?(jhuang)
blocking-b2g: 1.4? → backlog
(Reporter)

Comment 4

4 years ago
request to 1.5+
blocking-b2g: backlog → 1.5?

Comment 5

4 years ago
Wesley, who can help this in Gaia.
Flags: needinfo?(whuang)
Candice, is this in systemfe's plate?
Flags: needinfo?(cserran)

Updated

4 years ago
No longer blocks: 949293
(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)

Comment 9

4 years ago
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+
(Reporter)

Updated

4 years ago
Duplicate of this bug: 996507
Component: NFC → Gaia::System
(Assignee)

Updated

4 years ago
Assignee: nobody → gasolin
(Assignee)

Comment 15

4 years ago
Created attachment 8411552 [details] [review]
pull request redirect to github

@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)
(Reporter)

Comment 17

4 years ago
(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?
(Reporter)

Comment 19

4 years ago
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.
(Assignee)

Comment 20

4 years ago
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.
(Reporter)

Comment 22

4 years ago
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.
(Reporter)

Updated

4 years ago
Flags: needinfo?(allstars.chh)
Whiteboard: [systemsfe]
Whiteboard: [FT:System-Platform]
(Assignee)

Updated

4 years ago
Target Milestone: --- → 2.0 S1 (9may)
(Assignee)

Updated

4 years ago
Whiteboard: [FT:System-Platform] → [FT:System-Platform][p=1]
(Assignee)

Comment 23

4 years ago
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+
(Assignee)

Comment 25

4 years ago
add nfc_manager test case and merged to gaia-master https://github.com/mozilla-b2g/gaia/commit/ed00198def12a2df82fb8e4277fa5c3ed2471dd5

thanks!
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
status-b2g-v2.0: --- → fixed

Updated

4 years ago
Blocks: 1005797
Attachment #8411552 - Flags: feedback?(gweng)
(Assignee)

Updated

4 years ago
Blocks: 1023765
You need to log in before you can comment on or make changes to this bug.