Merge "Move the content description for Display size and text preview to xml" into main
This commit is contained in:
committed by
Android (Google) Code Review
commit
a9ef330701
@@ -104,13 +104,11 @@ class TextReadingPreviewController extends BasePreferenceController implements
|
||||
final boolean isLayoutRtl =
|
||||
origConfig.getLayoutDirection() == View.LAYOUT_DIRECTION_RTL;
|
||||
final int[] previewSamples = getPreviewSampleLayouts(mContext);
|
||||
final int[] previewContentDescriptions = getPreviewSampleContentDescriptions(mContext);
|
||||
final PreviewPagerAdapter pagerAdapter = new PreviewPagerAdapter(mContext, isLayoutRtl,
|
||||
previewSamples, createConfig(origConfig));
|
||||
mPreviewPreference.setPreviewAdapter(pagerAdapter);
|
||||
mPreviewPreference.setCurrentItem(
|
||||
isLayoutRtl ? previewSamples.length - 1 : FRAME_INITIAL_INDEX);
|
||||
mPreviewPreference.setContentDescription(previewContentDescriptions);
|
||||
|
||||
final int initialPagerIndex =
|
||||
mLastFontProgress * mDisplaySizeData.getValues().size() + mLastDisplayProgress;
|
||||
@@ -190,20 +188,6 @@ class TextReadingPreviewController extends BasePreferenceController implements
|
||||
return previewSamples;
|
||||
}
|
||||
|
||||
@VisibleForTesting(otherwise = VisibleForTesting.PRIVATE)
|
||||
static int[] getPreviewSampleContentDescriptions(Context context) {
|
||||
TypedArray typedArray = context.getResources().obtainTypedArray(
|
||||
R.array.config_text_reading_preview_content_descriptions);
|
||||
int previewCount = typedArray.length();
|
||||
int[] previewContentDescriptions = new int[previewCount];
|
||||
for (int i = 0; i < previewCount; i++) {
|
||||
previewContentDescriptions[i] =
|
||||
typedArray.getResourceId(i, R.string.preview_pager_content_description);
|
||||
}
|
||||
typedArray.recycle();
|
||||
return previewContentDescriptions;
|
||||
}
|
||||
|
||||
private int getPagerIndex() {
|
||||
final int displayDataSize = mDisplaySizeData.getValues().size();
|
||||
final int fontSizeProgress = mFontSizePreference.getProgress();
|
||||
|
||||
@@ -43,10 +43,26 @@ public class TextReadingPreviewPreference extends Preference {
|
||||
private int mCurrentItem;
|
||||
private int mLastLayerIndex;
|
||||
private PreviewPagerAdapter mPreviewAdapter;
|
||||
private int[] mContentDescriptions;
|
||||
|
||||
private int mLayoutMinHorizontalPadding = 0;
|
||||
private int mBackgroundMinHorizontalPadding = 0;
|
||||
private final ViewPager.OnPageChangeListener mPageChangeListener =
|
||||
new ViewPager.OnPageChangeListener() {
|
||||
@Override
|
||||
public void onPageScrolled(int i, float v, int i1) {
|
||||
// Do nothing
|
||||
}
|
||||
|
||||
@Override
|
||||
public void onPageSelected(int i) {
|
||||
mCurrentItem = i;
|
||||
}
|
||||
|
||||
@Override
|
||||
public void onPageScrollStateChanged(int i) {
|
||||
// Do nothing
|
||||
}
|
||||
};
|
||||
|
||||
TextReadingPreviewPreference(Context context) {
|
||||
super(context);
|
||||
@@ -78,23 +94,7 @@ public class TextReadingPreviewPreference extends Preference {
|
||||
adjustPaddings(previewLayout, backgroundView);
|
||||
|
||||
final ViewPager viewPager = (ViewPager) holder.findViewById(R.id.preview_pager);
|
||||
viewPager.addOnPageChangeListener(new ViewPager.OnPageChangeListener() {
|
||||
@Override
|
||||
public void onPageScrolled(int i, float v, int i1) {
|
||||
// Do nothing
|
||||
}
|
||||
|
||||
@Override
|
||||
public void onPageSelected(int i) {
|
||||
mCurrentItem = i;
|
||||
viewPager.setContentDescription(getContext().getString(mContentDescriptions[i]));
|
||||
}
|
||||
|
||||
@Override
|
||||
public void onPageScrollStateChanged(int i) {
|
||||
// Do nothing
|
||||
}
|
||||
});
|
||||
viewPager.addOnPageChangeListener(mPageChangeListener);
|
||||
final DotsPageIndicator pageIndicator =
|
||||
(DotsPageIndicator) holder.findViewById(R.id.page_indicator);
|
||||
updateAdapterIfNeeded(viewPager, pageIndicator, mPreviewAdapter);
|
||||
@@ -121,10 +121,6 @@ public class TextReadingPreviewPreference extends Preference {
|
||||
viewPager.setCurrentItem(getCurrentItem() + 1));
|
||||
nextButton.setContentDescription(getContext().getString(
|
||||
R.string.preview_pager_next_button));
|
||||
|
||||
// Initialize the content description since the OnPageChangeListener#onPageSelected won't
|
||||
// be called during setup.
|
||||
viewPager.setContentDescription(getContext().getString(mContentDescriptions[0]));
|
||||
}
|
||||
|
||||
@Override
|
||||
@@ -173,10 +169,6 @@ public class TextReadingPreviewPreference extends Preference {
|
||||
);
|
||||
}
|
||||
|
||||
void setContentDescription(int[] stringIds) {
|
||||
mContentDescriptions = stringIds;
|
||||
}
|
||||
|
||||
void setPreviewAdapter(PreviewPagerAdapter previewAdapter) {
|
||||
if (previewAdapter != mPreviewAdapter) {
|
||||
mPreviewAdapter = previewAdapter;
|
||||
|
||||
Reference in New Issue
Block a user