diff --git a/res/layout-land/wifi_dpp_qrcode_scanner_fragment.xml b/res/layout-land/wifi_dpp_qrcode_scanner_fragment.xml index fb50def7649..42178676acc 100644 --- a/res/layout-land/wifi_dpp_qrcode_scanner_fragment.xml +++ b/res/layout-land/wifi_dpp_qrcode_scanner_fragment.xml @@ -43,8 +43,5 @@ android:layout_height="wrap_content" android:layout_gravity="center"/> - - diff --git a/res/layout/wifi_dpp_activity.xml b/res/layout/wifi_dpp_activity.xml index a833dcc757c..cb82f661c13 100644 --- a/res/layout/wifi_dpp_activity.xml +++ b/res/layout/wifi_dpp_activity.xml @@ -22,12 +22,6 @@ android:layout_height="match_parent" android:orientation="vertical"> - - + android:layout_centerHorizontal="true" + android:gravity="center_horizontal" + android:orientation="vertical" + android:paddingStart="16dp" + android:paddingEnd="16dp"> - + - + android:gravity="center_horizontal" + android:textAlignment="center" + android:layout_marginTop="8dp" + android:paddingStart="32dp" + android:paddingEnd="32dp"/> + + diff --git a/src/com/android/settings/wifi/dpp/WifiDppQrCodeBaseFragment.java b/src/com/android/settings/wifi/dpp/WifiDppQrCodeBaseFragment.java index 5b76f11cdfa..6792dee7af9 100644 --- a/src/com/android/settings/wifi/dpp/WifiDppQrCodeBaseFragment.java +++ b/src/com/android/settings/wifi/dpp/WifiDppQrCodeBaseFragment.java @@ -43,6 +43,7 @@ import com.android.settings.R; * {@code WifiDppAddDeviceFragment} */ public abstract class WifiDppQrCodeBaseFragment extends InstrumentedFragment { + private ImageView mHeaderIcon; private TextView mTitle; private TextView mDescription; @@ -73,6 +74,7 @@ public abstract class WifiDppQrCodeBaseFragment extends InstrumentedFragment { } private void initView(View view) { + mHeaderIcon = view.findViewById(R.id.header_icon); mTitle = view.findViewById(R.id.title); mDescription = view.findViewById(R.id.description); mErrorMessage = view.findViewById(R.id.error_message); @@ -87,6 +89,10 @@ public abstract class WifiDppQrCodeBaseFragment extends InstrumentedFragment { mButtonRight = view.findViewById(R.id.button_right); } + protected void setHeaderIconImageResource(int resId) { + mHeaderIcon.setImageResource(resId); + } + protected void setTitle(String title) { mTitle.setText(title); } diff --git a/src/com/android/settings/wifi/dpp/WifiDppQrCodeGeneratorFragment.java b/src/com/android/settings/wifi/dpp/WifiDppQrCodeGeneratorFragment.java index ec22415d90b..81def9b339e 100644 --- a/src/com/android/settings/wifi/dpp/WifiDppQrCodeGeneratorFragment.java +++ b/src/com/android/settings/wifi/dpp/WifiDppQrCodeGeneratorFragment.java @@ -51,6 +51,7 @@ public class WifiDppQrCodeGeneratorFragment extends WifiDppQrCodeBaseFragment { public void onActivityCreated(Bundle savedInstanceState) { super.onActivityCreated(savedInstanceState); + setHeaderIconImageResource(R.drawable.ic_qrcode_24dp); WifiNetworkConfig wifiNetworkConfig = ((WifiNetworkConfig.Retriever) getActivity()) .getWifiNetworkConfig(); if (!WifiNetworkConfig.isValidConfig(wifiNetworkConfig)) { @@ -85,7 +86,7 @@ public class WifiDppQrCodeGeneratorFragment extends WifiDppQrCodeBaseFragment { @Override public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) { MenuItem item = menu.add(0, Menu.FIRST, 0, R.string.next_label); - item.setIcon(R.drawable.ic_menu_add); + item.setIcon(R.drawable.ic_scan_24dp); item.setShowAsAction(MenuItem.SHOW_AS_ACTION_ALWAYS); super.onCreateOptionsMenu(menu, inflater); diff --git a/src/com/android/settings/wifi/dpp/WifiDppQrCodeScannerFragment.java b/src/com/android/settings/wifi/dpp/WifiDppQrCodeScannerFragment.java index 64ede72ce43..8cd3c562cb3 100644 --- a/src/com/android/settings/wifi/dpp/WifiDppQrCodeScannerFragment.java +++ b/src/com/android/settings/wifi/dpp/WifiDppQrCodeScannerFragment.java @@ -89,6 +89,8 @@ public class WifiDppQrCodeScannerFragment extends WifiDppQrCodeBaseFragment impl public void onActivityCreated(Bundle savedInstanceState) { super.onActivityCreated(savedInstanceState); + setHeaderIconImageResource(R.drawable.ic_scan_24dp); + if (mConfiguratorMode) { setTitle(getString(R.string.wifi_dpp_add_device_to_network));