feat(new reviewer): hide answer buttons

xenonnn4w · Jul 18, 2024 · 4f47106 · 4f47106
1 parent 712f9fa
commit 4f47106
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 2 deletions.
diff --git a/AnkiDroid/src/main/java/com/ichi2/anki/ui/windows/reviewer/ReviewerFragment.kt b/AnkiDroid/src/main/java/com/ichi2/anki/ui/windows/reviewer/ReviewerFragment.kt
@@ -24,6 +24,7 @@ import android.view.Menu
 import android.view.MenuItem
 import android.view.View
 import android.webkit.WebView
+import android.widget.FrameLayout
 import androidx.activity.result.contract.ActivityResultContracts
 import androidx.annotation.StringRes
 import androidx.appcompat.view.menu.MenuBuilder
@@ -165,6 +166,12 @@ class ReviewerFragment :
     }
 
     private fun setupAnswerButtons(view: View) {
+        val hideAnswerButtons = sharedPrefs().getBoolean(getString(R.string.hide_answer_buttons_key), false)
+        if (hideAnswerButtons) {
+            view.findViewById<FrameLayout>(R.id.buttons_area).isVisible = false
+            return
+        }
+
         fun MaterialButton.setAnswerButtonNextTime(@StringRes title: Int, nextTime: String?) {
             val titleString = context.getString(title)
             text = ReviewerViewModel.buildAnswerButtonText(titleString, nextTime)

diff --git a/AnkiDroid/src/main/res/layout/reviewer2.xml b/AnkiDroid/src/main/res/layout/reviewer2.xml
@@ -11,7 +11,8 @@
 
     <androidx.constraintlayout.widget.ConstraintLayout
         android:layout_width="match_parent"
-        android:layout_height="match_parent">
+        android:layout_height="match_parent"
+        android:layout_marginBottom="4dp">
 
         <com.google.android.material.appbar.AppBarLayout
             android:id="@+id/appbar"
@@ -69,6 +70,7 @@
             android:layout_width="match_parent"
             android:layout_height="0dp"
             android:layout_marginHorizontal="8dp"
+            android:layout_marginBottom="4dp"
             app:layout_constraintTop_toBottomOf="@id/appbar"
             app:layout_constraintBottom_toTopOf="@id/buttons_area"
             style="@style/CardView.PreviewerStyle">
@@ -85,7 +87,7 @@
             android:id="@+id/buttons_area"
             android:layout_width="0dp"
             android:layout_height="@dimen/touch_target"
-            android:layout_marginVertical="2dp"
+            android:layout_marginTop="2dp"
             app:layout_constraintBottom_toBottomOf="parent"
             app:layout_constraintEnd_toEndOf="@id/webview_container"
             app:layout_constraintStart_toStartOf="@id/webview_container"

diff --git a/AnkiDroid/src/main/res/values/10-preferences.xml b/AnkiDroid/src/main/res/values/10-preferences.xml
@@ -443,4 +443,5 @@ this formatter is used if the bind only applies to both the question and the ans
     <string name="hide_system_bars_all_bars" comment="Setting option to hide all the system bars"
         >All</string>
     <string name="ignore_display_cutout" maxLength="41">Ignore display cutout</string>
+    <string name="hide_answer_buttons" maxLength="41">Hide answer buttons</string>
 </resources>
diff --git a/AnkiDroid/src/main/res/values/preferences.xml b/AnkiDroid/src/main/res/values/preferences.xml
@@ -203,4 +203,5 @@
     <!-- Reviewer options -->
     <string name="hide_system_bars_key">hideSystemBars</string>
     <string name="ignore_display_cutout_key">ignoreDisplayCutout</string>
+    <string name="hide_answer_buttons_key">hideAnswerButtons</string>
 </resources>
diff --git a/AnkiDroid/src/main/res/xml/preferences_reviewer.xml b/AnkiDroid/src/main/res/xml/preferences_reviewer.xml
@@ -15,4 +15,10 @@
         android:key="@string/ignore_display_cutout_key"
         android:title="@string/ignore_display_cutout"
         />
+
+    <SwitchPreferenceCompat
+        android:defaultValue="false"
+        android:key="@string/hide_answer_buttons_key"
+        android:title="@string/hide_answer_buttons"
+        />
 </androidx.preference.PreferenceScreen>