From 179dc3a1db672b345de8645ff55d3e659f6cf798 Mon Sep 17 00:00:00 2001
From: ajp_anton <ajp_anton@git.ajpanton.se>
Date: Sat, 29 Aug 2026 23:14:02 +0000
Subject: [PATCH] Fix Cards layout control IDs

---
 .../notificationsmaster/CardsLayoutFragment.kt       | 11 +++++++++--
 .../res/layout/include_cards_grid_settings_aod.xml   | 12 ++++++------
 .../include_cards_grid_settings_lockscreen.xml       | 12 ++++++------
 3 files changed, 21 insertions(+), 14 deletions(-)

diff --git a/app/src/main/java/se/ajpanton/notificationsmaster/CardsLayoutFragment.kt b/app/src/main/java/se/ajpanton/notificationsmaster/CardsLayoutFragment.kt
index a574bf0..43c03b5 100644
--- a/app/src/main/java/se/ajpanton/notificationsmaster/CardsLayoutFragment.kt
+++ b/app/src/main/java/se/ajpanton/notificationsmaster/CardsLayoutFragment.kt
@@ -113,9 +113,16 @@ class CardsLayoutFragment : Fragment(R.layout.fragment_cards_layout) {
     private fun controls(lockscreen: Boolean): Controls {
         val b = binding!!
         return if (lockscreen) b.lockscreenSettings.run {
-            Controls(maxRows, maxIcons, evenDistribution, limitWidth, iconHeight, iconHeightValue)
+            Controls(
+                lockscreenMaxRows,
+                lockscreenMaxIcons,
+                lockscreenEvenDistribution,
+                lockscreenLimitWidth,
+                lockscreenIconHeight,
+                lockscreenIconHeightValue,
+            )
         } else b.aodSettings.run {
-            Controls(maxRows, maxIcons, evenDistribution, limitWidth, iconHeight, iconHeightValue)
+            Controls(aodMaxRows, aodMaxIcons, aodEvenDistribution, aodLimitWidth, aodIconHeight, aodIconHeightValue)
         }
     }
 
diff --git a/app/src/main/res/layout/include_cards_grid_settings_aod.xml b/app/src/main/res/layout/include_cards_grid_settings_aod.xml
index f986a57..bacc3e4 100644
--- a/app/src/main/res/layout/include_cards_grid_settings_aod.xml
+++ b/app/src/main/res/layout/include_cards_grid_settings_aod.xml
@@ -5,14 +5,14 @@
     android:orientation="vertical">
 
     <TextView android:layout_width="match_parent" android:layout_height="wrap_content" android:layout_marginTop="12dp" android:text="Max rows" />
-    <Spinner android:id="@+id/max_rows" android:layout_width="wrap_content" android:layout_height="wrap_content" android:popupBackground="@drawable/dropdown_popup_background" />
+    <Spinner android:id="@+id/aod_max_rows" android:layout_width="wrap_content" android:layout_height="wrap_content" android:popupBackground="@drawable/dropdown_popup_background" />
     <TextView android:layout_width="match_parent" android:layout_height="wrap_content" android:layout_marginTop="8dp" android:text="Max icons per row" />
-    <Spinner android:id="@+id/max_icons" android:layout_width="wrap_content" android:layout_height="wrap_content" android:popupBackground="@drawable/dropdown_popup_background" />
-    <com.google.android.material.switchmaterial.SwitchMaterial android:id="@+id/even_distribution" android:layout_width="match_parent" android:layout_height="wrap_content" android:layout_marginTop="8dp" android:text="Even distribution" />
-    <com.google.android.material.switchmaterial.SwitchMaterial android:id="@+id/limit_width" android:layout_width="match_parent" android:layout_height="wrap_content" android:text="Limit to screen width" />
+    <Spinner android:id="@+id/aod_max_icons" android:layout_width="wrap_content" android:layout_height="wrap_content" android:popupBackground="@drawable/dropdown_popup_background" />
+    <com.google.android.material.switchmaterial.SwitchMaterial android:id="@+id/aod_even_distribution" android:layout_width="match_parent" android:layout_height="wrap_content" android:layout_marginTop="8dp" android:text="Even distribution" />
+    <com.google.android.material.switchmaterial.SwitchMaterial android:id="@+id/aod_limit_width" android:layout_width="match_parent" android:layout_height="wrap_content" android:text="Limit to screen width" />
     <LinearLayout android:layout_width="match_parent" android:layout_height="wrap_content" android:layout_marginTop="8dp" android:gravity="center_vertical" android:orientation="horizontal">
         <TextView android:layout_width="0dp" android:layout_height="wrap_content" android:layout_weight="1" android:text="Icon height (% of status bar)" />
-        <TextView android:id="@+id/icon_height_value" android:layout_width="wrap_content" android:layout_height="wrap_content" />
+        <TextView android:id="@+id/aod_icon_height_value" android:layout_width="wrap_content" android:layout_height="wrap_content" />
     </LinearLayout>
-    <SeekBar android:id="@+id/icon_height" android:layout_width="match_parent" android:layout_height="wrap_content" />
+    <SeekBar android:id="@+id/aod_icon_height" android:layout_width="match_parent" android:layout_height="wrap_content" />
 </LinearLayout>
diff --git a/app/src/main/res/layout/include_cards_grid_settings_lockscreen.xml b/app/src/main/res/layout/include_cards_grid_settings_lockscreen.xml
index f986a57..0829ff5 100644
--- a/app/src/main/res/layout/include_cards_grid_settings_lockscreen.xml
+++ b/app/src/main/res/layout/include_cards_grid_settings_lockscreen.xml
@@ -5,14 +5,14 @@
     android:orientation="vertical">
 
     <TextView android:layout_width="match_parent" android:layout_height="wrap_content" android:layout_marginTop="12dp" android:text="Max rows" />
-    <Spinner android:id="@+id/max_rows" android:layout_width="wrap_content" android:layout_height="wrap_content" android:popupBackground="@drawable/dropdown_popup_background" />
+    <Spinner android:id="@+id/lockscreen_max_rows" android:layout_width="wrap_content" android:layout_height="wrap_content" android:popupBackground="@drawable/dropdown_popup_background" />
     <TextView android:layout_width="match_parent" android:layout_height="wrap_content" android:layout_marginTop="8dp" android:text="Max icons per row" />
-    <Spinner android:id="@+id/max_icons" android:layout_width="wrap_content" android:layout_height="wrap_content" android:popupBackground="@drawable/dropdown_popup_background" />
-    <com.google.android.material.switchmaterial.SwitchMaterial android:id="@+id/even_distribution" android:layout_width="match_parent" android:layout_height="wrap_content" android:layout_marginTop="8dp" android:text="Even distribution" />
-    <com.google.android.material.switchmaterial.SwitchMaterial android:id="@+id/limit_width" android:layout_width="match_parent" android:layout_height="wrap_content" android:text="Limit to screen width" />
+    <Spinner android:id="@+id/lockscreen_max_icons" android:layout_width="wrap_content" android:layout_height="wrap_content" android:popupBackground="@drawable/dropdown_popup_background" />
+    <com.google.android.material.switchmaterial.SwitchMaterial android:id="@+id/lockscreen_even_distribution" android:layout_width="match_parent" android:layout_height="wrap_content" android:layout_marginTop="8dp" android:text="Even distribution" />
+    <com.google.android.material.switchmaterial.SwitchMaterial android:id="@+id/lockscreen_limit_width" android:layout_width="match_parent" android:layout_height="wrap_content" android:text="Limit to screen width" />
     <LinearLayout android:layout_width="match_parent" android:layout_height="wrap_content" android:layout_marginTop="8dp" android:gravity="center_vertical" android:orientation="horizontal">
         <TextView android:layout_width="0dp" android:layout_height="wrap_content" android:layout_weight="1" android:text="Icon height (% of status bar)" />
-        <TextView android:id="@+id/icon_height_value" android:layout_width="wrap_content" android:layout_height="wrap_content" />
+        <TextView android:id="@+id/lockscreen_icon_height_value" android:layout_width="wrap_content" android:layout_height="wrap_content" />
     </LinearLayout>
-    <SeekBar android:id="@+id/icon_height" android:layout_width="match_parent" android:layout_height="wrap_content" />
+    <SeekBar android:id="@+id/lockscreen_icon_height" android:layout_width="match_parent" android:layout_height="wrap_content" />
 </LinearLayout>