Make the whole notifications permission banner clickable

wordpress-mobile · Apr 6, 2023 · 63431aa · 63431aa
1 parent ddbac77
commit 63431aa
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/WordPress/src/main/java/org/wordpress/android/ui/notifications/NotificationsListFragment.kt b/WordPress/src/main/java/org/wordpress/android/ui/notifications/NotificationsListFragment.kt
@@ -217,15 +217,14 @@ class NotificationsListFragment : Fragment(R.layout.notifications_list_fragment)
             viewModel.resetNotificationsPermissionWarningDismissState()
         }
 
-        if (Build.VERSION.SDK_INT < Build.VERSION_CODES.TIRAMISU ||
-            hasPermission ||
+        if (Build.VERSION.SDK_INT < Build.VERSION_CODES.TIRAMISU || hasPermission ||
             viewModel.isNotificationsPermissionsWarningDismissed
         ) {
             // If the user dismissed the warning, don't show it again.
             notificationPermissionWarning.isVisible = false
         } else {
             notificationPermissionWarning.isVisible = true
-            permissionFixButton.setOnClickListener {
+            notificationPermissionWarning.setOnClickListener {
                 val isAlwaysDenied = WPPermissionUtils.isPermissionAlwaysDenied(
                     requireActivity(),
                     Manifest.permission.POST_NOTIFICATIONS

diff --git a/WordPress/src/main/res/layout/notifications_list_fragment.xml b/WordPress/src/main/res/layout/notifications_list_fragment.xml
@@ -57,10 +57,8 @@
                 android:textColor="?attr/colorOnSurface" />
 
             <com.google.android.material.textview.MaterialTextView
-                android:id="@+id/permission_fix_button"
                 android:layout_width="wrap_content"
                 android:layout_height="@dimen/min_touch_target_sz"
-                android:background="?selectableItemBackgroundBorderless"
                 android:gravity="center"
                 android:minWidth="@dimen/min_touch_target_sz"
                 android:text="@string/notifications_permission_fix"