Closed
Bug 923088
Opened 11 years ago
Closed 11 years ago
[Settings] Change network mode names to make them user friendly and consistent with previous OS versions
Categories
(Firefox OS Graveyard :: Gaia::Settings, defect)
Tracking
(blocking-b2g:koi+, b2g-v1.2 verified)
People
(Reporter: brg, Assigned: gasolin)
References
Details
(Whiteboard: burirun2)
Attachments
(2 files)
Tested using Unagi, v1.2, Moz RIL, Gecko-b318b86.Gaia-5e0d0df.
In Settings -> Cellular&Data -> Network operator -> Network type
Current options for WCDMA devices are:
wcdma/gsm
gsm
wcdma
Proposed translations already used in v1.1/v1.0.1:
Automatic
2G only
3G only
Updated•11 years ago
|
Whiteboard: burirun2
Reporter | ||
Comment 1•11 years ago
|
||
Certification blocker for launching v1.2. Nominating accordingly.
blocking-b2g: --- → koi?
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → gasolin
Comment 3•11 years ago
|
||
This issue was introduced by supporting CDMA in v1.2. A phone may support combination of network modes. Please consult UX for the detail strings of the combination.
Comment 5•11 years ago
|
||
So, Is this certification blocker? This is not an obvious issue.
Because if we want to consider any one of these names. Like 3G, 3.5G, Edge, GPRS, GSM and Dual SIM..We will have lots of options to use. I think it's depends on partner's requirement. What do you think?
Flags: needinfo?(nhsieh)
Comment 6•11 years ago
|
||
This is related to bug 910111. All possible options are:
wcdma/gsm
gsm
wcdma
wcdma/gsm-auto
cdma/evdo
cdma
evdo
wcdma/gsm/cdma/evdo
We should consider all possible combination (i.e. devices support GSM, CDMA, or GSM + CDMA) based on requirements and define proper strings for them.
Reporter | ||
Comment 7•11 years ago
|
||
This issue is related to bug 903232.
When network modes will be displayed accordingly to device capabilities(bug 903232), then we need this bug to fix that WCDMA modes change to the same names that we use in the v1-train:
wcdma/gsm --> Automatic
gsm --> 2G only
wcdma --> 3G only
wcdma/gsm-auto --> Honestly, I do not know the difference between wcdma/gsm and wcdma/gsm-auto, Can you describe it?
In regard of the other network capabilities(CDMA), we did not use it before so I do not have official translation, we can track them in bug 910111 as you suggested and keep this bug for CDMA ones that we already used in v1-train.
Assignee | ||
Comment 8•11 years ago
|
||
wcdma/gsm-auto is a kind of wcdma/gsm but gsm is more perfered.
Assignee | ||
Comment 9•11 years ago
|
||
Reference
http://android.stackexchange.com/questions/28553/change-wcdma-to-gsm-automatically
since bug 903232 will hide unrelated CDMA types, we could focus on provide these 4 types with user friednly name.
in our context, I suggest to use
wcdma/gsm --> 3G preffered (left 'Automatic' for DSDS wcdma/gsm/cdma/evdo mode)
gsm --> 2G only
wcdma --> 3G only
wcdma/gsm-auto --> 2G preffered
Neo, how do you think?
Flags: needinfo?(nhsieh)
Assignee | ||
Comment 10•11 years ago
|
||
Attachment #816910 -
Flags: review?(arthur.chen)
Assignee | ||
Comment 11•11 years ago
|
||
Comment 12•11 years ago
|
||
Comment on attachment 816910 [details]
pull request redirect to github
Thanks for the patch! Please check my comments in github.
And I would suggest to detect the capability of the device (using getSupportedNetworkInfo in settings/utils.js) and assign corresponding strings. For GSM devices, we keep using Automatic, 2G only, 3G only, 2G preferred. For CDMA devices or GSM + CDMA devices, we can use the terms as the reference provided in comment 9. By doing this we don't need to update the strings when we enable CDMA features in the future.
Attachment #816910 -
Flags: review?(arthur.chen)
Comment 13•11 years ago
|
||
Is there going to be a way to test those strings if they're device-specific?
Assignee | ||
Comment 14•11 years ago
|
||
Comment on attachment 816910 [details]
pull request redirect to github
Use three maps (GSM only, CDMA only, Dual stack) detected by result.gsm/result.cdma
GSM only, CDMA only mode will follow [Automatic, 2G only, 3G only] pattern.
Dual stack mode will use [GSM, CDMA(UPPER words), GSM preffered.., Automatic] pattern
ask for review again
Attachment #816910 -
Flags: review?(arthur.chen)
Comment 15•11 years ago
|
||
Comment on attachment 816910 [details]
pull request redirect to github
r=me with the github comments addressed. Thanks for the patch!
Attachment #816910 -
Flags: review?(arthur.chen) → review+
Comment 16•11 years ago
|
||
Axel, I think we can mock different devices by tweaking some variable in the unit tests.
Assignee | ||
Comment 17•11 years ago
|
||
merged in gaia-master
https://github.com/mozilla-b2g/gaia/commit/7c6a7b1bfb8a33ccf8af64c83c5100f26eb211c1
thanks!
we can use build-time customization in network.json file to put any order of network modes
Status: NEW → RESOLVED
Closed: 11 years ago
status-b2g-v1.2:
--- → affected
Flags: needinfo?(nhsieh)
Resolution: --- → FIXED
Comment 18•11 years ago
|
||
Uplifted 7c6a7b1bfb8a33ccf8af64c83c5100f26eb211c1 to:
v1.2: fc6ee255e10c7ec1762fe1f54f28d01fb7aa0f94
Comment 19•11 years ago
|
||
Verified Fixed on Aurora 1.2 Buri and Leo devices. The Network Type displays: WCDMA preferred, GSM, WCDMA, GSM preferred, EVDO preferred, CDMA, EVDO, Automatic.
- Leo 1.2 -
Gaia dbcc171eae6b4d9d168a48291d5ea54c7580561a
SourceStamp 7ca5d1e81d37
BuildID 20131017004001
Version 26.0a2
- Buri 1.2 -
Gaia dbcc171eae6b4d9d168a48291d5ea54c7580561a
SourceStamp 7ca5d1e81d37
BuildID 20131017004001
Version 26.0a2
Comment 20•11 years ago
|
||
Wasabi 1.2
Gaia mozillaorg/v1.2 - 1fd315337d8ae891c3024e4c682c4c50797ea40e
Gecko mozillaorg/v1.2 - 66bc401ea42298f15a24e38042ed4886e10b9382
The network type displays: Automatic, 2G only, 3G only
build with gaia/distribution/
network.json
============
{
"types": [
"cdma/evdo",
"cdma",
"evdo"
]
}
settings.json
=============
{
"ril.radio.preferredNetworkType":"cdma/evdo"
}
Comment 21•11 years ago
|
||
(In reply to Askeing Yen[:askeing] from comment #20)
> Wasabi 1.2
> Gaia mozillaorg/v1.2 - 1fd315337d8ae891c3024e4c682c4c50797ea40e
> Gecko mozillaorg/v1.2 - 66bc401ea42298f15a24e38042ed4886e10b9382
>
> The network type displays: Automatic, 2G only, 3G only
https://github.com/mozilla-b2g/gaia/blob/master/apps/settings/js/carrier.js#L752
https://github.com/mozilla-b2g/gaia/blob/master/apps/settings/locales/settings.en-US.properties#L172
Verified on 1.2 wasabi.
You need to log in
before you can comment on or make changes to this bug.
Description
•