From b7a18c7bdce32bfc7656f196534a386645b1cac8 Mon Sep 17 00:00:00 2001
From: Aayush Gupta <aayushgupta219@gmail.com>
Date: Wed, 1 Jan 2025 14:24:38 +0700
Subject: [PATCH] fixup! SettingsFragment: Add preference for permissions
 fragment

Signed-off-by: Aayush Gupta <aayushgupta219@gmail.com>
---
 .../view/ui/onboarding/PermissionsFragment.kt |  5 +-
 .../fragment_onboarding_permissions.xml       | 78 ++++++++++++-------
 2 files changed, 50 insertions(+), 33 deletions(-)

diff --git a/app/src/main/java/com/aurora/store/view/ui/onboarding/PermissionsFragment.kt b/app/src/main/java/com/aurora/store/view/ui/onboarding/PermissionsFragment.kt
index 41ad637cb..8211c0618 100644
--- a/app/src/main/java/com/aurora/store/view/ui/onboarding/PermissionsFragment.kt
+++ b/app/src/main/java/com/aurora/store/view/ui/onboarding/PermissionsFragment.kt
@@ -57,9 +57,8 @@ class PermissionsFragment : BaseFragment<FragmentOnboardingPermissionsBinding>()
         super.onViewCreated(view, savedInstanceState)
 
         // Headers are only visible if we are onboarding
-        binding.title?.isVisible = args.isOnboarding
-        binding.subtitle?.isVisible = args.isOnboarding
-        binding.toolbar?.apply {
+        binding.title.isVisible = args.isOnboarding
+        binding.toolbar.apply {
             isVisible = !args.isOnboarding
             setNavigationOnClickListener { findNavController().navigateUp() }
         }
diff --git a/app/src/main/res/layout-land/fragment_onboarding_permissions.xml b/app/src/main/res/layout-land/fragment_onboarding_permissions.xml
index f08b3278f..34513659e 100644
--- a/app/src/main/res/layout-land/fragment_onboarding_permissions.xml
+++ b/app/src/main/res/layout-land/fragment_onboarding_permissions.xml
@@ -20,43 +20,61 @@
   -->
 
 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
+    android:orientation="vertical"
     android:showDividers="middle">
 
+    <androidx.appcompat.widget.Toolbar
+        android:id="@+id/toolbar"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:minHeight="?attr/actionBarSize"
+        android:visibility="gone"
+        app:navigationIcon="@drawable/ic_arrow_back"
+        app:title="@string/onboarding_title_permissions" />
+
     <LinearLayout
         android:layout_width="match_parent"
-        android:layout_height="match_parent"
-        android:layout_weight="1"
-        android:gravity="center"
-        android:orientation="vertical">
+        android:layout_height="wrap_content">
 
-        <androidx.appcompat.widget.AppCompatTextView
-            android:layout_width="wrap_content"
-            android:layout_height="wrap_content"
-            android:includeFontPadding="true"
-            android:text="@string/onboarding_title_permissions"
-            android:textAlignment="textStart"
-            android:textColor="?colorAccent"
-            android:textSize="42sp" />
+        <LinearLayout
+            android:layout_width="match_parent"
+            android:layout_height="match_parent"
+            android:layout_weight="1"
+            android:gravity="center"
+            android:orientation="vertical">
 
-        <androidx.appcompat.widget.AppCompatTextView
-            style="@style/AuroraTextStyle.Subtitle.Alt"
-            android:layout_width="wrap_content"
-            android:layout_height="wrap_content"
-            android:text="@string/onboarding_permission_select"
-            android:textAlignment="textStart" />
+            <androidx.appcompat.widget.AppCompatTextView
+                android:id="@+id/title"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:includeFontPadding="true"
+                android:text="@string/onboarding_title_permissions"
+                android:textAlignment="textStart"
+                android:textColor="?colorAccent"
+                android:textSize="42sp" />
+
+            <androidx.appcompat.widget.AppCompatTextView
+                android:id="@+id/subtitle"
+                style="@style/AuroraTextStyle.Subtitle.Alt"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:text="@string/onboarding_permission_select"
+                android:textAlignment="textStart" />
+        </LinearLayout>
+
+        <com.airbnb.epoxy.EpoxyRecyclerView
+            android:id="@+id/epoxy_recycler"
+            android:layout_width="match_parent"
+            android:layout_height="match_parent"
+            android:layout_weight="1"
+            android:clipToPadding="false"
+            android:overScrollMode="never"
+            android:padding="@dimen/padding_medium"
+            android:scrollbars="none"
+            tools:listitem="@layout/view_permission" />
     </LinearLayout>
-
-    <com.airbnb.epoxy.EpoxyRecyclerView
-        android:id="@+id/epoxy_recycler"
-        android:layout_width="match_parent"
-        android:layout_height="match_parent"
-        android:layout_weight="1"
-        android:clipToPadding="false"
-        android:overScrollMode="never"
-        android:padding="@dimen/padding_medium"
-        android:scrollbars="none"
-        tools:listitem="@layout/view_permission" />
-</LinearLayout>
\ No newline at end of file
+</LinearLayout>