chore: changing actions in reader fragment.

## Types of changes

- [ ] I have read the **CONTRIBUTING** document.
- [ ] My code follows the code style of this project.
- [ ] I have updated the documentation accordingly.
- [ ] I have added tests to cover my changes.
- [ ] All new and existing tests passed.
- [ ] This is **NOT** translation related.

This closes issue #XXX

This is implements feature #YYY

This finishes chore #ZZZ

Reviewed-on: #176
Co-authored-by: Amine <amine.bouabdallaoui@pm.me>
Co-committed-by: Amine <amine.bouabdallaoui@pm.me>

CHANGELOG.md

@@ -1,3 +1,11 @@
+**v125010131
+
+- fix: reload the adapter when it's needed. Fixes #128. (#176)
+- feat: basic auth and images loading. Fixes #172. (#175)
+- Changelog for v125010111
+
+--------------------------------------------------------------------
+
 **v125010111
 
 - Debug trying to fix context issues. (#174)

androidApp/src/main/java/bou/amine/apps/readerforselfossv2/android/HomeActivity.kt

@@ -317,13 +317,9 @@ class HomeActivity :
 
     private fun reloadLayoutManager() {
         val currentManager = binding.recyclerView.layoutManager
-        val layoutManager: RecyclerView.LayoutManager
 
-        // This will only update the layout manager if settings changed
-        when (currentManager) {
-            is StaggeredGridLayoutManager ->
-                if (!appSettingsService.isCardViewEnabled()) {
-                    layoutManager =
+        fun gridLayoutManager() {
+            val layoutManager =
                 GridLayoutManager(
                     this,
                     calculateNoOfColumns(),
@@ -331,9 +327,8 @@ class HomeActivity :
             binding.recyclerView.layoutManager = layoutManager
         }
 
-            is GridLayoutManager ->
-                if (appSettingsService.isCardViewEnabled()) {
-                    layoutManager =
+        fun staggererdGridLayoutManager() {
+            var layoutManager =
                 StaggeredGridLayoutManager(
                     calculateNoOfColumns(),
                     StaggeredGridLayoutManager.VERTICAL,
@@ -343,24 +338,23 @@ class HomeActivity :
             binding.recyclerView.layoutManager = layoutManager
         }
 
+        when (currentManager) {
+            is StaggeredGridLayoutManager ->
+                if (!appSettingsService.isCardViewEnabled()) {
+                    gridLayoutManager()
+                }
+
+            is GridLayoutManager ->
+                if (appSettingsService.isCardViewEnabled()) {
+                    staggererdGridLayoutManager()
+                }
+
             else ->
                 if (currentManager == null) {
                     if (!appSettingsService.isCardViewEnabled()) {
-                        layoutManager =
-                            GridLayoutManager(
-                                this,
-                                calculateNoOfColumns(),
-                            )
-                        binding.recyclerView.layoutManager = layoutManager
+                        gridLayoutManager()
                     } else {
-                        layoutManager =
-                            StaggeredGridLayoutManager(
-                                calculateNoOfColumns(),
-                                StaggeredGridLayoutManager.VERTICAL,
-                            )
-                        layoutManager.gapStrategy =
-                            StaggeredGridLayoutManager.GAP_HANDLING_MOVE_ITEMS_BETWEEN_SPANS
-                        binding.recyclerView.layoutManager = layoutManager
+                        staggererdGridLayoutManager()
                     }
                 }
         }
@@ -485,8 +479,8 @@ class HomeActivity :
     }
 
     private fun handleListResult(appendResults: Boolean = false) {
-        if (appendResults) {
         val oldManager = binding.recyclerView.layoutManager
+        if (appendResults) {
             firstVisible =
                 when (oldManager) {
                     is StaggeredGridLayoutManager ->
@@ -499,7 +493,13 @@ class HomeActivity :
                 }
         }
 
-        if (recyclerAdapter == null) {
+        @Suppress("detekt:ComplexCondition")
+        if (recyclerAdapter == null ||
+            (
+                (recyclerAdapter is ItemListAdapter && appSettingsService.isCardViewEnabled()) ||
+                    (recyclerAdapter is ItemCardAdapter && !appSettingsService.isCardViewEnabled())
+            )
+        ) {
             if (appSettingsService.isCardViewEnabled()) {
                 recyclerAdapter =
                     ItemCardAdapter(

fastlane/metadata/android/en-US/changelogs/v125010131.txt

@@ -0,0 +1,5 @@
+**v125010131**
+
+- fix: reload the adapter when it's needed. Fixes #128. (#176)
+- feat: basic auth and images loading. Fixes #172. (#175)
+- Changelog for v125010111