Closed Bug 1072125 Opened 10 years ago Closed 10 years ago

[Settings] label and carrier name in "Network operators in the area" out of place

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-b2g:2.1+, b2g-v2.0 unaffected, b2g-v2.1 verified, b2g-v2.2 verified)

VERIFIED FIXED
2.1 S5 (26sep)
blocking-b2g 2.1+
Tracking Status
b2g-v2.0 --- unaffected
b2g-v2.1 --- verified
b2g-v2.2 --- verified

People

(Reporter: ericcc, Assigned: eragonj)

References

Details

(Whiteboard: [p=1])

Attachments

(3 files)

Attached image 2014-09-24 14.11.05.jpg
### STR
1. Settings -> Cellular & DAta -> Network Operator
2. Turn offf Automatical selection
3. Tap Search again to get a list of operators

### Actual
See 2014-09-24 14.11.05.jpg

### Version
Gaia-Rev        b542080231bb9cdbdb282ab926965ad3f91f7460
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-aurora/rev/d614ae398c03
Build-ID        20140923155123
Version         34.0a2
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  27
FW-Date         Thu Sep  4 14:59:02 CST 2014
Bootloader      L1TC10011800
[Blocking Requested - why for this release]: UI Regression
blocking-b2g: --- → 2.1?
QA Whiteboard: [COM=Gaia::Settings]
Flags: needinfo?(ejchen)
Attached image SettingsManual.jpg
Assignee: nobody → ejchen
Flags: needinfo?(ejchen)
Whiteboard: [p=1]
Target Milestone: --- → 2.1 S6 (10oct)
Attached file patch on master (2.2)
Arthur, can you help me review this small patch ? THanks :)
Attachment #8494941 - Flags: review?(arthur.chen)
Comment on attachment 8494941 [details] [review]
patch on master (2.2)

Looks good to me, thanks!
Attachment #8494941 - Flags: review?(arthur.chen) → review+
Thanks all, this patch was merged into Gaia/master (2.2) : https://github.com/mozilla-b2g/gaia/commit/278577d42e016eedf320a906c36df069b1e564e8
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Triage: regression, blocking.
blocking-b2g: 2.1? → 2.1+
Comment on attachment 8494941 [details] [review]
patch on master (2.2)

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #):
[User impact] if declined: Users would get confused because UI is wrong.
[Testing completed]: no
[Risk to taking this patch] (and alternatives if risky): low
[String changes made]: no
Attachment #8494941 - Flags: approval-gaia-v2.1?
Attachment #8494941 - Flags: approval-gaia-v2.1? → approval-gaia-v2.1+
This issue is verified on Flame 2.2 and 2.1:

Flame 2.2 KitKat Base (319mb)(Full Flash)
  
Environmental Variables:
Device: Flame 2.2 Master
BuildID: 20141002093155
Gaia: 191d805f4911628d37a8a90a1e23a6013995138f
Gecko: 5d6ec4dddf14
Gonk: 2c909e821d107d414f851e267dedcd7aae2cebf
Version: 35.0a1 (2.2 Master)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:35.0) Gecko/35.0 Firefox/35.0
  
Flame 2.1 KitKat Base (319mb)(Full Flash)

Environmental Variables:
Device: Flame 2.1
BuildID: 20141002000202
Gaia: 94dcc25f2e34a4900ea58310c26be52bcb089161
Gecko: baaa0c3ab8fd
Gonk: 2c909e821d107d414f851e267dedcd7aae2cebf
Version: 34.0a2 (2.1)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

The operators and status are displayed properly.
Status: RESOLVED → VERIFIED
QA Whiteboard: [COM=Gaia::Settings] → [COM=Gaia::Settings][QAnalyst-Triage?]
Flags: needinfo?(ktucker)
QA Whiteboard: [COM=Gaia::Settings][QAnalyst-Triage?] → [COM=Gaia::Settings][QAnalyst-Triage+]
Flags: needinfo?(ktucker)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: