Move the text under slider bar to the description under title
- Make this change after discussed with UX team. Fixes: 206737771 Test: manual test Change-Id: I9517933312d8db400f6d40b4124815ad275014ec
This commit is contained in:
@@ -26,6 +26,7 @@
|
||||
android:layout_height="wrap_content">
|
||||
|
||||
<LinearLayout
|
||||
android:id="@+id/font_size_preview_text_group"
|
||||
android:layout_width="match_parent"
|
||||
android:layout_height="match_parent"
|
||||
android:paddingTop="16dp"
|
||||
|
@@ -75,12 +75,6 @@
|
||||
android:contentDescription="@string/font_size_make_larger_desc"
|
||||
style="@style/screen_size_imageview_style"/>
|
||||
</LinearLayout>
|
||||
|
||||
<TextView
|
||||
android:layout_width="wrap_content"
|
||||
android:layout_height="wrap_content"
|
||||
android:text="@string/font_size_summary"
|
||||
android:textAppearance="@*android:style/TextAppearance.DeviceDefault.Subhead"/>
|
||||
</LinearLayout>
|
||||
</LinearLayout>
|
||||
</androidx.core.widget.NestedScrollView>
|
||||
|
@@ -74,12 +74,6 @@
|
||||
android:contentDescription="@string/screen_zoom_make_larger_desc"
|
||||
style="@style/screen_size_imageview_style"/>
|
||||
</LinearLayout>
|
||||
|
||||
<TextView
|
||||
android:layout_width="wrap_content"
|
||||
android:layout_height="wrap_content"
|
||||
android:text="@string/screen_zoom_summary"
|
||||
android:textAppearance="@*android:style/TextAppearance.DeviceDefault.Subhead"/>
|
||||
</LinearLayout>
|
||||
</LinearLayout>
|
||||
</androidx.core.widget.NestedScrollView>
|
||||
|
@@ -19,4 +19,6 @@
|
||||
<!-- Dashboard number of columns -->
|
||||
<integer name="dashboard_num_columns">2</integer>
|
||||
|
||||
<!-- Whether to support large screen -->
|
||||
<bool name="config_supported_large_screen">true</bool>
|
||||
</resources>
|
||||
|
@@ -63,6 +63,9 @@
|
||||
<dimen name="confirm_credentials_side_margin">0dp</dimen>
|
||||
<dimen name="confirm_credentials_top_margin">64dp</dimen>
|
||||
|
||||
<!-- padding for font size preview in large screen -->
|
||||
<dimen name="font_size_preview_padding_start">32dp</dimen>
|
||||
|
||||
<!-- Padding for screen pinning -->
|
||||
<dimen name="screen_pinning_padding_start">40dp</dimen>
|
||||
<dimen name="screen_pinning_padding_end">40dp</dimen>
|
||||
|
@@ -532,6 +532,9 @@
|
||||
<!-- Whether suw to support two panes -->
|
||||
<bool name="config_suw_supported_two_panes">false</bool>
|
||||
|
||||
<!-- Whether to support large screen -->
|
||||
<bool name="config_supported_large_screen">false</bool>
|
||||
|
||||
<!-- Display settings screen, Color mode options. Must be the same length and order as
|
||||
config_color_mode_options_values below. Only the values that also appear in
|
||||
frameworks/base/core/res/res/values/config.xml's config_availableColorModes are shown. -->
|
||||
|
@@ -120,8 +120,8 @@ public class AccessibilityScreenSizeForSetupWizardActivity extends InstrumentedA
|
||||
: R.string.screen_zoom_title);
|
||||
((TextView) findViewById(R.id.sud_layout_subtitle)).setText(
|
||||
getFragmentType(getIntent()) == FragmentType.FONT_SIZE
|
||||
? R.string.short_summary_font_size
|
||||
: R.string.screen_zoom_short_summary);
|
||||
? R.string.font_size_summary
|
||||
: R.string.screen_zoom_summary);
|
||||
}
|
||||
|
||||
private boolean isSuwSupportedTwoPanes() {
|
||||
|
@@ -17,6 +17,13 @@
|
||||
package com.android.settings.display;
|
||||
|
||||
import android.app.settings.SettingsEnums;
|
||||
import android.os.Bundle;
|
||||
import android.view.LayoutInflater;
|
||||
import android.view.View;
|
||||
import android.view.ViewGroup;
|
||||
import android.widget.LinearLayout;
|
||||
|
||||
import androidx.viewpager.widget.ViewPager;
|
||||
|
||||
import com.android.settings.R;
|
||||
|
||||
@@ -33,6 +40,23 @@ public class FontSizePreferenceFragmentForSetupWizard
|
||||
return SettingsEnums.SUW_ACCESSIBILITY_FONT_SIZE;
|
||||
}
|
||||
|
||||
@Override
|
||||
public View onCreateView(LayoutInflater inflater, ViewGroup container,
|
||||
Bundle savedInstanceState) {
|
||||
final View root = super.onCreateView(inflater, container, savedInstanceState);
|
||||
if (getResources().getBoolean(R.bool.config_supported_large_screen)) {
|
||||
final ViewPager viewPager = root.findViewById(R.id.preview_pager);
|
||||
final View view = (View) viewPager.getAdapter().instantiateItem(viewPager,
|
||||
viewPager.getCurrentItem());
|
||||
final LinearLayout layout = view.findViewById(R.id.font_size_preview_text_group);
|
||||
final int paddingStart = getResources().getDimensionPixelSize(
|
||||
R.dimen.font_size_preview_padding_start);
|
||||
layout.setPaddingRelative(paddingStart, layout.getPaddingTop(),
|
||||
layout.getPaddingEnd(), layout.getPaddingBottom());
|
||||
}
|
||||
return root;
|
||||
}
|
||||
|
||||
@Override
|
||||
public void onStop() {
|
||||
// Log the final choice in value if it's different from the previous value.
|
||||
|
Reference in New Issue
Block a user