From 55e9b175638af49546f144c6195cd579a9cb793b Mon Sep 17 00:00:00 2001
From: Ammar Githam <ammargitham786@gmail.com>
Date: Sun, 4 Jul 2021 02:57:42 +0900
Subject: [PATCH] Remove 'Disable animation' from Posts layout preferences as
 it is not used

---
 .../PostsLayoutPreferencesDialogFragment.java |  6 ---
 .../layout/dialog_post_layout_preferences.xml | 52 +++++--------------
 2 files changed, 13 insertions(+), 45 deletions(-)

diff --git a/app/src/main/java/awais/instagrabber/dialogs/PostsLayoutPreferencesDialogFragment.java b/app/src/main/java/awais/instagrabber/dialogs/PostsLayoutPreferencesDialogFragment.java
index 9fc1fa6c..9e35ae54 100644
--- a/app/src/main/java/awais/instagrabber/dialogs/PostsLayoutPreferencesDialogFragment.java
+++ b/app/src/main/java/awais/instagrabber/dialogs/PostsLayoutPreferencesDialogFragment.java
@@ -80,7 +80,6 @@ public class PostsLayoutPreferencesDialogFragment extends DialogFragment {
         initAvatarsToggle();
         initCornersToggle();
         initGapToggle();
-        initAnimationDisableToggle();
     }
 
     private void initLayoutToggle() {
@@ -170,11 +169,6 @@ public class PostsLayoutPreferencesDialogFragment extends DialogFragment {
         binding.showGapToggle.setOnCheckedChangeListener((buttonView, isChecked) -> preferencesBuilder.setHasGap(isChecked));
     }
 
-    private void initAnimationDisableToggle() {
-        binding.disableAnimationToggle.setChecked(preferencesBuilder.isAnimationDisabled());
-        binding.disableAnimationToggle.setOnCheckedChangeListener((buttonView, isChecked) -> preferencesBuilder.setAnimationDisabled(isChecked));
-    }
-
     private int getSelectedLayoutId() {
         switch (preferencesBuilder.getType()) {
             case STAGGERED_GRID:
diff --git a/app/src/main/res/layout/dialog_post_layout_preferences.xml b/app/src/main/res/layout/dialog_post_layout_preferences.xml
index b53b0f78..57e1f201 100644
--- a/app/src/main/res/layout/dialog_post_layout_preferences.xml
+++ b/app/src/main/res/layout/dialog_post_layout_preferences.xml
@@ -23,8 +23,8 @@
             android:id="@+id/label_layout"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
-            android:layout_marginRight="6dp"
             android:layout_marginTop="4dp"
+            android:layout_marginRight="6dp"
             android:layout_marginBottom="4dp"
             android:gravity="right"
             android:text="@string/layout_style"
@@ -74,14 +74,14 @@
             android:id="@+id/label_col_count"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
-            android:layout_marginRight="6dp"
             android:layout_marginTop="4dp"
+            android:layout_marginRight="6dp"
             android:layout_marginBottom="4dp"
             android:gravity="right"
             android:text="@string/column_count"
-            app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintBottom_toBottomOf="@id/col_count_toggle"
             app:layout_constraintEnd_toStartOf="@id/guideline"
+            app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toTopOf="@id/col_count_toggle" />
 
         <com.google.android.material.button.MaterialButtonToggleGroup
@@ -116,8 +116,8 @@
             android:id="@+id/label_show_names_toggle"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
-            android:layout_marginRight="6dp"
             android:layout_marginTop="4dp"
+            android:layout_marginRight="6dp"
             android:layout_marginBottom="4dp"
             android:gravity="right"
             android:text="@string/show_names"
@@ -142,14 +142,14 @@
             android:id="@+id/label_show_avatar_toggle"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
-            android:layout_marginRight="6dp"
             android:layout_marginTop="4dp"
+            android:layout_marginRight="6dp"
             android:layout_marginBottom="4dp"
             android:gravity="right"
             android:text="@string/show_avatars"
-            app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintBottom_toBottomOf="@id/show_avatar_toggle"
             app:layout_constraintEnd_toStartOf="@id/guideline"
+            app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toTopOf="@id/show_avatar_toggle" />
 
         <com.google.android.material.switchmaterial.SwitchMaterial
@@ -168,14 +168,14 @@
             android:id="@+id/label_avatar_size"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
-            android:layout_marginRight="6dp"
             android:layout_marginTop="4dp"
+            android:layout_marginRight="6dp"
             android:layout_marginBottom="4dp"
             android:gravity="right"
             android:text="@string/avatar_size"
-            app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintBottom_toBottomOf="@id/avatar_size_toggle"
             app:layout_constraintEnd_toStartOf="@id/guideline"
+            app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toTopOf="@id/avatar_size_toggle" />
 
         <com.google.android.material.button.MaterialButtonToggleGroup
@@ -217,14 +217,14 @@
             android:id="@+id/label_corners"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
-            android:layout_marginRight="6dp"
             android:layout_marginTop="4dp"
+            android:layout_marginRight="6dp"
             android:layout_marginBottom="4dp"
             android:gravity="right"
             android:text="@string/corners"
-            app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintBottom_toBottomOf="@id/corners_toggle"
             app:layout_constraintEnd_toStartOf="@id/guideline"
+            app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toTopOf="@id/corners_toggle" />
 
         <com.google.android.material.button.MaterialButtonToggleGroup
@@ -259,14 +259,14 @@
             android:id="@+id/label_gap"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
-            android:layout_marginRight="6dp"
             android:layout_marginTop="4dp"
+            android:layout_marginRight="6dp"
             android:layout_marginBottom="4dp"
             android:gravity="right"
             android:text="@string/show_grid_gap"
-            app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintBottom_toBottomOf="@id/show_gap_toggle"
             app:layout_constraintEnd_toStartOf="@id/guideline"
+            app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toTopOf="@id/show_gap_toggle" />
 
         <com.google.android.material.switchmaterial.SwitchMaterial
@@ -274,39 +274,13 @@
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:checked="true"
-            app:layout_constraintBottom_toTopOf="@id/disable_animation_toggle"
+            app:layout_constraintBottom_toTopOf="@id/show_gap_toggle"
             app:layout_constraintStart_toEndOf="@id/guideline"
             app:layout_constraintTop_toBottomOf="@id/corners_toggle"
             app:showText="false"
             app:switchPadding="0dp"
             app:thumbTextPadding="0dp" />
 
-        <androidx.appcompat.widget.AppCompatTextView
-            android:id="@+id/label_disable_animation"
-            android:layout_width="0dp"
-            android:layout_height="wrap_content"
-            android:layout_marginRight="6dp"
-            android:layout_marginTop="4dp"
-            android:layout_marginBottom="4dp"
-            android:gravity="right"
-           android:text="@string/disable_animation"
-            android:layout_gravity="right"
-            app:layout_constraintStart_toStartOf="parent"
-            app:layout_constraintBottom_toBottomOf="@id/disable_animation_toggle"
-            app:layout_constraintEnd_toStartOf="@id/guideline"
-            app:layout_constraintTop_toTopOf="@id/disable_animation_toggle" />
-
-        <com.google.android.material.switchmaterial.SwitchMaterial
-            android:id="@+id/disable_animation_toggle"
-            android:layout_width="wrap_content"
-            android:layout_height="wrap_content"
-            android:checked="false"
-            app:layout_constraintStart_toEndOf="@id/guideline"
-            app:layout_constraintTop_toBottomOf="@id/show_gap_toggle"
-            app:showText="false"
-            app:switchPadding="0dp"
-            app:thumbTextPadding="0dp" />
-
         <androidx.constraintlayout.widget.Group
             android:id="@+id/staggered_or_grid_options"
             android:layout_width="wrap_content"