Smoothen skimming with page slider

avluis · Feb 26, 2023 · 2ce290c · 2ce290c
1 parent 62969d0
commit 2ce290c
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/app/src/main/java/me/devsaki/hentoid/fragments/reader/ReaderNavigation.java b/app/src/main/java/me/devsaki/hentoid/fragments/reader/ReaderNavigation.java
@@ -181,14 +181,17 @@ void updatePageControls() {
         pageMaxNumber.setText(String.format(Locale.ENGLISH, "%d", maxPageNum));
         superBinding.viewerPagenumberText.setText(String.format(Locale.ENGLISH, "%d / %d", pageNum, maxPageNum));
 
-        int sliderMaxPos = Math.max(1, maxPageNum - 1);
-        // Next line to avoid setting a max position inferior to current position
-        binding.pageSlider.setValue(Helper.coerceIn(binding.pageSlider.getValue(), 0, sliderMaxPos));
-        binding.pageSlider.setValueTo(sliderMaxPos);
-
-        int imageIndex = getCurrentImageIndex();
-        if (imageIndex > -1)
-            binding.pageSlider.setValue(Helper.coerceIn(imageIndex, 0, sliderMaxPos));
+        // Only update slider when user isn't skimming _with_ the slider
+        if (binding.imagePreviewCenter.getVisibility() != View.VISIBLE) {
+            int sliderMaxPos = Math.max(1, maxPageNum - 1);
+            // Next line to avoid setting a max position inferior to current position
+            binding.pageSlider.setValue(Helper.coerceIn(binding.pageSlider.getValue(), 0, sliderMaxPos));
+            binding.pageSlider.setValueTo(sliderMaxPos);
+
+            int imageIndex = getCurrentImageIndex();
+            if (imageIndex > -1)
+                binding.pageSlider.setValue(Helper.coerceIn(imageIndex, 0, sliderMaxPos));
+        }
 
         if (!Preferences.isReaderChapteredNavigation() || null == chapters || chapters.isEmpty()) {
             prevFunctionalButton.setVisibility(isContentFirst ? View.INVISIBLE : View.VISIBLE);