From 700d8de5bef9c80342099b4ef072ad37efb75a1a Mon Sep 17 00:00:00 2001
From: Ebrahim Byagowi <ebrahim@gnu.org>
Date: Fri, 1 May 2020 19:37:19 +0430
Subject: Fix PagerIndicatorView disabled dot in RTL languages

---
 .../danoeh/antennapod/view/PagerIndicatorView.java | 22 +++++++++++++++-------
 1 file changed, 15 insertions(+), 7 deletions(-)

(limited to 'app/src/main/java/de')

diff --git a/app/src/main/java/de/danoeh/antennapod/view/PagerIndicatorView.java b/app/src/main/java/de/danoeh/antennapod/view/PagerIndicatorView.java
index 240a565c8..10ed98769 100644
--- a/app/src/main/java/de/danoeh/antennapod/view/PagerIndicatorView.java
+++ b/app/src/main/java/de/danoeh/antennapod/view/PagerIndicatorView.java
@@ -23,6 +23,7 @@ public class PagerIndicatorView extends View {
     private int disabledPage = -1;
     private int circleColor = 0;
     private int circleColorHighlight = -1;
+    private boolean isLocaleRtl = false;
 
     public PagerIndicatorView(Context context) {
         super(context);
@@ -40,6 +41,9 @@ public class PagerIndicatorView extends View {
     }
 
     private void setup() {
+        isLocaleRtl = TextUtilsCompat.getLayoutDirectionFromLocale(Locale.getDefault())
+                == ViewCompat.LAYOUT_DIRECTION_RTL;
+
         paint.setAntiAlias(true);
         paint.setStyle(Paint.Style.FILL);
 
@@ -50,6 +54,14 @@ public class PagerIndicatorView extends View {
         a.recycle();
     }
 
+    /**
+     * Visual and logical position distinction only happens in RTL locales (e.g. Persian)
+     * where pages positions are flipped thus it does nothing in LTR locales (e.g. English)
+     */
+    private float logicalPositionToVisual(float position) {
+        return isLocaleRtl ? numPages - 1 - position : position;
+    }
+
     public void setViewPager(ViewPager2 pager) {
         numPages = pager.getAdapter().getItemCount();
         pager.getAdapter().registerAdapterDataObserver(new RecyclerView.AdapterDataObserver() {
@@ -59,22 +71,18 @@ public class PagerIndicatorView extends View {
                 invalidate();
             }
         });
-        boolean isLocaleRtl = TextUtilsCompat.getLayoutDirectionFromLocale(Locale.getDefault())
-                == ViewCompat.LAYOUT_DIRECTION_RTL;
         pager.registerOnPageChangeCallback(new ViewPager2.OnPageChangeCallback() {
             @Override
             public void onPageScrolled(int position, float positionOffset, int positionOffsetPixels) {
-                PagerIndicatorView.this.position = position + positionOffset;
-                if (isLocaleRtl) {
-                    PagerIndicatorView.this.position = numPages - 1 - PagerIndicatorView.this.position;
-                }
+                PagerIndicatorView.this.position = logicalPositionToVisual(
+                        position + positionOffset);
                 invalidate();
             }
         });
     }
 
     public void setDisabledPage(int disabledPage) {
-        this.disabledPage = disabledPage;
+        this.disabledPage = (int) logicalPositionToVisual(disabledPage);
         invalidate();
     }
 
-- 
cgit v1.2.3