Closed Bug 943227 Opened 11 years ago Closed 11 years ago

[DSDS] When device is out of coverage, only SIM 2 icon on status bar is correct.

Categories

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

x86_64
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-b2g:1.3+, firefox29 fixed, b2g-v1.3 verified)

VERIFIED FIXED
1.3 C2/1.4 S2(17jan)
blocking-b2g 1.3+
Tracking Status
firefox29 --- fixed
b2g-v1.3 --- verified

People

(Reporter: echu, Assigned: arthurcc)

References

Details

(Whiteboard: [dsds_US_test])

Attachments

(2 files)

When device is out of coverage, only SIM 2 icon on status bar is correct while SIM 1 icon is only grey bars without red "x" on it. Please refer to UX spec in bug 917705.

* Build Number                
Gaia:     671a7c1fae1c32a77b2675636c7efdf089fd6443
Gecko:    b79ea773754854e17566f302e80e2654d9c155c9
BuildID   20131126092608
Version   28.0a1

* Reproduce Steps
Let device enter no network coverage area. Check RSSI icon on status bar.

* Expected Result
Both SIM should have correct icon.

* Actual Result
SIM 2 icon is correct, but SIM 1 icon is only grey bars without red "x" on it.

* Occurrence rate
100%
blocking-b2g: --- → 1.3?
Whiteboard: [FT:RIL]
Whiteboard: [FT:RIL] → [dsds_US_test]
Summary: [fugu] When device is out of coverage, only SIM 2 icon on status bar is correct. → [fugu][DSDS] When device is out of coverage, only SIM 2 icon on status bar is correct.
blocking-b2g: 1.3? → 1.3+
Arthur, can you help this one?
Flags: needinfo?(arthur.chen)
Summary: [fugu][DSDS] When device is out of coverage, only SIM 2 icon on status bar is correct. → [DSDS] When device is out of coverage, only SIM 2 icon on status bar is correct.
Assignee: nobody → arthur.chen
Flags: needinfo?(arthur.chen)
Target Milestone: --- → 1.3 C2/1.4 S2(17jan)
See Also: → 938440
Component: Gaia → Gaia::System
I found that the `voice.emergencyCallsOnly` property of the first sim slot is true, so that we won't display a red `x` on it. The red `x` means we don't have any signal and unavailable for emergency calls. The same behavior can be observed on single sim devices.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
Hi Carrie,

Do you agree with comment 2? If so, could you help to update RSSI part of DSDS 1.3 spec?

Thanks.
Flags: needinfo?(cawang)
I'd like to know the single SIM situation. If it's out of coverage on single SIM, do we show the icon with the x? If so, then dual SIM should definitely follow the same rule. Thanks!
Flags: needinfo?(cawang)
We don't show the icon if it is still able to make emergency calls.
(In reply to cawang from comment #4)
> I'd like to know the single SIM situation. If it's out of coverage on single
> SIM, do we show the icon with the x? If so, then dual SIM should definitely
> follow the same rule. Thanks!

Hi Carrie,

For single SIM, there is no "x" icon as well. Sorry that I have to ni? you again to confirm if this is acceptable, could you help to update RSSI spec for DSDS?
Flags: needinfo?(cawang)
Hi Enpei,

yes, we think it shall be consistent that both single SIM and DSDS shall show "x" when it's out of coverage.
Thanks!
Flags: needinfo?(cawang)
Based on UX comment 7, reopen the bug.
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
After confirmed with Edgar and Hsin-yi, the behavior of the properties are as the following:

- When voice.connected is true, emergencyCallsOnly is always false, and voice.state is always "registered".
- When voice.connected is false, emergencyCallsOnly is always true, and voice.state must not equal to "registered".

Per an offline discussion with Carrie, we agree with the following behavior:

- If the device is connected to a network (voice.connected is true), show the normal signal bars based on the signal strength (even if it is zero).
- If the device is not connected to a network (voice.connected is false), show the signal bars with a red "x".
Per comment 9, we don't need to check emergencyCallsOnly when voice.connected is false. In this patch I remove the emergencyCallsOnly attribute and related logic and styles.

Alive, could you help review the patch? Thanks!
Attachment #8366561 - Flags: review?(alive)
release triage assessment: fix for 1.3
Attachment #8366561 - Flags: review?(alive) → review+
master: 04c6d23125dd52b1ca4b9c33be1344ffe33e8c5d
Status: REOPENED → RESOLVED
Closed: 11 years ago11 years ago
Resolution: --- → FIXED
Uplifted 04c6d23125dd52b1ca4b9c33be1344ffe33e8c5d to:
v1.3: ab86d5a6920e278bdc106751a2dcdeb1f93b9cf8
Verified. 

Fugu
Gaia      bb36b4164d3e51ca04b612e507e1178f80bf240d
Gecko     4227240a4d3e1e418ced160c2a48cb454d7b5645
BuildID   20140204104742
Version   28.0
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: