Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Bug 1123164 - [Settings]There is no SIM settings icons when you insert SIM2 only in Settings -> Cellular & Data. #27515

Merged
merged 1 commit into from Jan 29, 2015

Conversation

mpizza
Copy link
Contributor

@mpizza mpizza commented Jan 20, 2015

No description provided.

@try-server-hook
Copy link

mpizza GaryChen, PYChen, Gary (mpizza) started tests. Results

break;
default:
if (pendingTargetPanel) {
SettingsService.navigate(pendingTargetPanel);
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: Let's still add a break here.

@mpizza mpizza force-pushed the bug_1123164_dsds_issue branch 4 times, most recently from d135ea8 to 53f22fc Compare January 21, 2015 08:36
@try-server-hook
Copy link

mpizza GaryChen, PYChen (mpizza) started tests. Results

@try-server-hook
Copy link

mpizza GaryChen, PYChen (mpizza) started tests. Results

@try-server-hook
Copy link

mpizza GaryChen, PYChen (mpizza) started tests. Results

@@ -5,6 +5,7 @@ require(['config/require'], function() {
// The following are the scripts used by many other scripts. We load them
// at once here. These should be move to the dependency of each panel in the
// future.
var DsdsSettings = require('dsds_settings');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: Please move this to the line under var Settings = require('settings');.

@try-server-hook
Copy link

mpizza GaryChen, PYChen (mpizza) started tests. Results

@try-server-hook
Copy link

mpizza GaryChen, PYChen (mpizza) started tests. Results

@try-server-hook
Copy link

mpizza GaryChen, PYChen (mpizza) started tests. Results

@try-server-hook
Copy link

mpizza GaryChen, PYChen (mpizza) started tests. Results

@try-server-hook
Copy link

mpizza GaryChen, PYChen (mpizza) started tests. Results

@try-server-hook
Copy link

mpizza GaryChen, PYChen (mpizza) started tests. Results

@try-server-hook
Copy link

mpizza GaryChen, PYChen (mpizza) started tests. Results

…t SIM2 only in Settings -> Cellular & Data. r=arthurcc.
@try-server-hook
Copy link

mpizza GaryChen, PYChen (mpizza) started tests. Results

mpizza added a commit that referenced this pull request Jan 29, 2015
Bug 1123164 - [Settings]There is no SIM settings icons when you insert SIM2 only in Settings -> Cellular & Data. r=arthurcc.
@mpizza mpizza merged commit 5a6e46a into mozilla-b2g:master Jan 29, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants