Revert "Left & Right buttons for Display size & text preview"
This reverts commit 30e63f6178
.
Reason for revert: Can cause crash for some users?
Change-Id: I4b4e71157b668bbf4320a610ad14ddd10f2eade3
This commit is contained in:
committed by
Android (Google) Code Review
parent
30e63f6178
commit
b1691756e2
@@ -22,7 +22,6 @@ import android.os.Parcelable;
|
||||
import android.util.AttributeSet;
|
||||
import android.view.View;
|
||||
import android.widget.FrameLayout;
|
||||
import android.widget.ImageButton;
|
||||
import android.widget.LinearLayout;
|
||||
|
||||
import androidx.preference.Preference;
|
||||
@@ -100,23 +99,6 @@ public class TextReadingPreviewPreference extends Preference {
|
||||
(DotsPageIndicator) holder.findViewById(R.id.page_indicator);
|
||||
updateAdapterIfNeeded(viewPager, pageIndicator, mPreviewAdapter);
|
||||
updatePagerAndIndicator(viewPager, pageIndicator);
|
||||
viewPager.setClipToOutline(true);
|
||||
|
||||
int layoutDirection =
|
||||
getContext().getResources().getConfiguration().getLayoutDirection();
|
||||
int previousId = (layoutDirection == View.LAYOUT_DIRECTION_RTL)
|
||||
? R.id.preview_right_button : R.id.preview_left_button;
|
||||
int nextId = (layoutDirection == View.LAYOUT_DIRECTION_RTL)
|
||||
? R.id.preview_left_button : R.id.preview_right_button;
|
||||
final ImageButton previousButton = previewLayout.findViewById(previousId);
|
||||
final ImageButton nextButton = previewLayout.findViewById(nextId);
|
||||
|
||||
previousButton.setOnClickListener((view) -> setCurrentItem(getCurrentItem() - 1));
|
||||
previousButton.setContentDescription(getContext().getString(
|
||||
R.string.preview_pager_previous_button));
|
||||
nextButton.setOnClickListener((view) -> setCurrentItem(getCurrentItem() + 1));
|
||||
previousButton.setContentDescription(getContext().getString(
|
||||
R.string.preview_pager_next_button));
|
||||
}
|
||||
|
||||
@Override
|
||||
@@ -176,9 +158,7 @@ public class TextReadingPreviewPreference extends Preference {
|
||||
Preconditions.checkNotNull(mPreviewAdapter,
|
||||
"Preview adapter is null, you should init the preview adapter first");
|
||||
|
||||
if (currentItem < 0 || currentItem >= mPreviewAdapter.getCount()) {
|
||||
return;
|
||||
} else if (currentItem != mCurrentItem) {
|
||||
if (currentItem != mCurrentItem) {
|
||||
mCurrentItem = currentItem;
|
||||
notifyChanged();
|
||||
}
|
||||
|
Reference in New Issue
Block a user