Merge "[Manual Network Selection] UX improvement"
This commit is contained in:
@@ -135,6 +135,9 @@ public class NetworkSelectSettings extends DashboardFragment {
|
|||||||
super.onStart();
|
super.onStart();
|
||||||
|
|
||||||
updateForbiddenPlmns();
|
updateForbiddenPlmns();
|
||||||
|
if (isProgressBarVisible()) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
setProgressBarVisible(true);
|
setProgressBarVisible(true);
|
||||||
|
|
||||||
mNetworkScanHelper.startNetworkScan(
|
mNetworkScanHelper.startNetworkScan(
|
||||||
@@ -157,7 +160,6 @@ public class NetworkSelectSettings extends DashboardFragment {
|
|||||||
@Override
|
@Override
|
||||||
public void onStop() {
|
public void onStop() {
|
||||||
super.onStop();
|
super.onStop();
|
||||||
stopNetworkQuery();
|
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
@@ -377,6 +379,13 @@ public class NetworkSelectSettings extends DashboardFragment {
|
|||||||
mConnectedPreferenceCategory.setVisible(true);
|
mConnectedPreferenceCategory.setVisible(true);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private boolean isProgressBarVisible() {
|
||||||
|
if (mProgressHeader == null) {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
return (mProgressHeader.getVisibility() == View.VISIBLE);
|
||||||
|
}
|
||||||
|
|
||||||
protected void setProgressBarVisible(boolean visible) {
|
protected void setProgressBarVisible(boolean visible) {
|
||||||
if (mProgressHeader != null) {
|
if (mProgressHeader != null) {
|
||||||
mProgressHeader.setVisibility(visible ? View.VISIBLE : View.GONE);
|
mProgressHeader.setVisibility(visible ? View.VISIBLE : View.GONE);
|
||||||
@@ -425,6 +434,7 @@ public class NetworkSelectSettings extends DashboardFragment {
|
|||||||
|
|
||||||
@Override
|
@Override
|
||||||
public void onDestroy() {
|
public void onDestroy() {
|
||||||
|
stopNetworkQuery();
|
||||||
mNetworkScanExecutor.shutdown();
|
mNetworkScanExecutor.shutdown();
|
||||||
super.onDestroy();
|
super.onDestroy();
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user