feat: Implement adaptive Master-Detail layout for Settings

- Refactor Settings screen into a Master-Detail architecture using `ListDetailPaneScaffold`.
- Create `SettingsCategoriesViewModel` and `SettingsMasterScreen` to manage category selection (Theme, Security, Info).
- Move detailed preference logic to `SettingsDetailScreen` and separate components into `ThemePreferences`, `SecurityPreferences`, and `InfoPreferences`.
- Update `AdaptiveInteractor` and `SettingsViewModel` to synchronize category selection state across adaptive panes.
- Add pull-to-refresh support to both settings master and detail views for state re-sync.
- Introduce `SettingsCategory` enum in the domain module to standardize category identification.
- Optimize adaptive UI components by extracting `VerticalPaneExpansionDragHandle` and `selectedListItemColor` into shared composables.
- Update `Router` and its implementations to better handle the lifecycle and navigation of the adaptive navigator.
- Enhance UI tests and test screens to accommodate the new settings navigation flow and categories.
- Improve build efficiency by updating the `build_quick.sh` script to more accurately exclude redundant iOS-related Gradle tasks.
- Disable the `ui:test` CocoaPods release framework for iOS simulators to reduce unnecessary linking during test execution.

Author: softartdev

.github/workflows/kmp.yml

@@ -36,7 +36,9 @@ jobs:
         with:
           path: ~/.konan
           key: kotlin-${{ steps.kotlin-version.outputs.version }}
-      - name: Build
+      - name: Quick build
+        run: ./gradle/build_quick.sh
+      - name: Full build
         run: ./gradlew build
       - name: Archive build-output artifacts
         if: always()

CONTRIBUTING.md

@@ -225,6 +225,7 @@ private fun ScreenContent(
 4. **Preview functions**: Add `@Preview` for visual components
 5. **Remember wisely**: Use `remember` for expensive calculations
 6. **Keys in lists**: Provide stable keys for LazyColumn/LazyRow
+7. **Adaptive drag handle**: Prefer method references for pane expansion, e.g. `paneExpansionDragHandle = ThreePaneScaffoldScope::VerticalPaneExpansionDragHandle` (use a lambda in Kotlin/Wasm actuals to avoid the current composable function reference compiler issue).
 
 ### ViewModel Style
 

app/android/src/androidTest/java/com/softartdev/notedelight/ui/SignInToSettingsTest.kt

@@ -34,6 +34,9 @@ class SignInToSettingsTest {
     private val composeUiTest: ComposeUiTest = reflect(composeTestRule)
 
     @Test
-    fun signInToSettingsTest() = SignInToSettingsTestCase(composeUiTest, Espresso::closeSoftKeyboard).invoke()
+    fun signInToSettingsTest() = SignInToSettingsTestCase(
+        composeUiTest,
+        Espresso::closeSoftKeyboard,
+        Espresso::pressBack,
+    ).invoke()
 }
-

app/desktop/src/jvmTest/kotlin/com/softartdev/notedelight/ui/DesktopUiTests.kt

@@ -9,7 +9,7 @@ import androidx.compose.ui.test.ExperimentalTestApi
 import androidx.compose.ui.test.assertIsDisplayed
 import androidx.compose.ui.test.junit4.ComposeContentTestRule
 import androidx.compose.ui.test.junit4.createComposeRule
-import androidx.compose.ui.test.onNodeWithContentDescription
+import androidx.compose.ui.test.onAllNodesWithContentDescription
 import androidx.compose.ui.test.performClick
 import androidx.lifecycle.Lifecycle
 import androidx.lifecycle.compose.LocalLifecycleOwner
@@ -104,9 +104,12 @@ class DesktopUiTests : AbstractJvmUiTests() {
     override fun localeTest() = super.localeTest()
 
     override fun pressBack() {
-        composeTestRule.onNodeWithContentDescription(label = Icons.AutoMirrored.Filled.ArrowBack.name)
-            .assertIsDisplayed()
-            .performClick()
+        val backButtons = composeTestRule.onAllNodesWithContentDescription(
+            label = Icons.AutoMirrored.Filled.ArrowBack.name
+        )
+        val nodes = backButtons.fetchSemanticsNodes()
+        val rightMostIndex = nodes.indices.maxBy { index -> nodes[index].boundsInRoot.left }
+        backButtons[rightMostIndex].assertIsDisplayed().performClick()
     }
 
     override fun closeSoftKeyboard() = Unit // Desktop doesn't have soft keyboard

app/iosApp/Pods/Pods.xcodeproj/project.pbxproj

@@ -0,0 +1,13 @@
+package com.softartdev.notedelight.model
+
+enum class SettingsCategory {
+    Theme,
+    Security,
+    Info;
+
+    val id: Long = ordinal.toLong()
+
+    companion object {
+        fun fromId(id: Long?): SettingsCategory? = id?.toInt()?.let(entries::get)
+    }
+}

app/iosApp/iosApp.xcworkspace/xcuserdata/artur.xcuserdatad/UserInterfaceState.xcuserstate

@@ -56,6 +56,7 @@ Each screen has a dedicated ViewModel following **MVVM pattern**:
 
 #### Settings
 - `SettingsViewModel`: App settings management
+- `SettingsCategoriesViewModel`: Category selection and adaptive navigation for settings
 - `SecurityResult`: Security settings states
 - **Password Management:**
   - `EnterViewModel`: Enter new password
@@ -68,7 +69,7 @@ Each screen has a dedicated ViewModel following **MVVM pattern**:
 
 ### Interactors (`interactor/`)
 
-- `AdaptiveInteractor`: Shared state holder that bridges adaptive navigation between `MainViewModel`, router, and UI panes. It also exposes `checkSaveChangeChannel` so the list pane can request a save-change confirmation before switching or creating notes in adaptive layouts.
+- `AdaptiveInteractor`: Shared state holder that bridges adaptive navigation between `MainViewModel`, `SettingsCategoriesViewModel`, router, and UI panes. It also exposes `selectedSettingsCategoryIdStateFlow` for settings selection sync and `checkSaveChangeChannel` so the list pane can request a save-change confirmation before switching or creating notes in adaptive layouts.
 - `SnackbarInteractor`: Multiplatform contract used by ViewModels to emit UI messages without depending on Compose APIs.
 
 ### Navigation (`navigation/`)

core/domain/src/commonMain/kotlin/com/softartdev/notedelight/model/SettingsCategory.kt

@@ -8,19 +8,29 @@ import com.softartdev.notedelight.CoroutineDispatchersStub
 import com.softartdev.notedelight.PrintLogWriter
 import com.softartdev.notedelight.db.NoteDAO
 import com.softartdev.notedelight.interactor.AdaptiveInteractor
+import com.softartdev.notedelight.interactor.LocaleInteractor
 import com.softartdev.notedelight.interactor.SnackbarInteractor
+import com.softartdev.notedelight.model.SettingsCategory
 import com.softartdev.notedelight.model.Note
 import com.softartdev.notedelight.navigation.Router
 import com.softartdev.notedelight.presentation.MainDispatcherRule
 import com.softartdev.notedelight.presentation.main.MainAction
 import com.softartdev.notedelight.presentation.main.MainViewModel
 import com.softartdev.notedelight.presentation.main.NoteListResult
 import com.softartdev.notedelight.presentation.note.NoteAction
+import com.softartdev.notedelight.presentation.note.NoteResult
 import com.softartdev.notedelight.presentation.note.NoteViewModel
+import com.softartdev.notedelight.presentation.settings.SecurityResult
+import com.softartdev.notedelight.presentation.settings.SettingsAction
+import com.softartdev.notedelight.presentation.settings.SettingsCategoriesAction
+import com.softartdev.notedelight.presentation.settings.SettingsCategoriesViewModel
+import com.softartdev.notedelight.presentation.settings.SettingsViewModel
 import com.softartdev.notedelight.repository.SafeRepo
 import com.softartdev.notedelight.usecase.note.CreateNoteUseCase
 import com.softartdev.notedelight.usecase.note.DeleteNoteUseCase
 import com.softartdev.notedelight.usecase.note.SaveNoteUseCase
+import com.softartdev.notedelight.usecase.crypt.CheckSqlCipherVersionUseCase
+import com.softartdev.notedelight.usecase.settings.RevealFileListUseCase
 import com.softartdev.notedelight.util.createLocalDateTime
 import kotlinx.coroutines.ExperimentalCoroutinesApi
 import kotlinx.coroutines.async
@@ -52,11 +62,16 @@ class AdaptiveInteractorTest {
     private val mockCreateNoteUseCase = Mockito.mock(CreateNoteUseCase::class.java)
     private val mockDeleteNoteUseCase = Mockito.mock(DeleteNoteUseCase::class.java)
     private val mockSnackbarInteractor = Mockito.mock(SnackbarInteractor::class.java)
+    private val mockLocaleInteractor = Mockito.mock(LocaleInteractor::class.java)
+    private val checkSqlCipherVersionUseCase = CheckSqlCipherVersionUseCase(mockSafeRepo)
+    private val revealFileListUseCase = RevealFileListUseCase()
     private val adaptiveInteractor = AdaptiveInteractor()
     private val coroutineDispatchers = CoroutineDispatchersStub(mainDispatcherRule.testDispatcher)
-    
+
     private lateinit var mainViewModel: MainViewModel
     private lateinit var noteViewModel: NoteViewModel
+    private lateinit var settingsCategoriesViewModel: SettingsCategoriesViewModel
+    private lateinit var settingsViewModel: SettingsViewModel
 
     private val id = 1L
     private val title: String = "title"
@@ -84,6 +99,19 @@ class AdaptiveInteractorTest {
             router = mockRouter,
             coroutineDispatchers = coroutineDispatchers
         )
+        settingsCategoriesViewModel = SettingsCategoriesViewModel(
+            router = mockRouter,
+            adaptiveInteractor = adaptiveInteractor,
+        )
+        settingsViewModel = SettingsViewModel(
+            safeRepo = mockSafeRepo,
+            checkSqlCipherVersionUseCase = checkSqlCipherVersionUseCase,
+            snackbarInteractor = mockSnackbarInteractor,
+            router = mockRouter,
+            revealFileListUseCase = revealFileListUseCase,
+            localeInteractor = mockLocaleInteractor,
+            adaptiveInteractor = adaptiveInteractor,
+        )
         Mockito.`when`(mockNoteDAO.pagingDataFlow).thenReturn(flowOf(PagingData.empty()))
         Mockito.`when`(mockSafeRepo.noteDAO).thenReturn(mockNoteDAO)
         Mockito.`when`(mockNoteDAO.count()).thenReturn(0)
@@ -93,7 +121,7 @@ class AdaptiveInteractorTest {
 
     @After
     fun tearDown() = runTest {
-        Mockito.reset(mockSafeRepo, mockRouter, mockNoteDAO, mockCreateNoteUseCase, mockDeleteNoteUseCase, mockSnackbarInteractor)
+        Mockito.reset(mockSafeRepo, mockRouter, mockNoteDAO, mockCreateNoteUseCase, mockDeleteNoteUseCase, mockSnackbarInteractor, mockLocaleInteractor)
         Logger.setLogWriters()
     }
 
@@ -103,20 +131,20 @@ class AdaptiveInteractorTest {
         noteViewModel.launchCollectingSelectedNoteId()
 
         mainViewModel.stateFlow.test {
-            val initialResult = awaitItem()
+            val initialResult: NoteListResult = awaitItem()
             assertTrue(initialResult is NoteListResult.Success)
             assertNull(initialResult.selectedId)
 
             mainViewModel.onAction(MainAction.OnNoteClick(id))
 
-            val updatedResult = awaitItem()
+            val updatedResult: NoteListResult = awaitItem()
             assertTrue(updatedResult is NoteListResult.Success)
             assertEquals(id, updatedResult.selectedId)
 
             cancelAndIgnoreRemainingEvents()
         }
         noteViewModel.stateFlow.test {
-            var noteResult = awaitItem()
+            var noteResult: NoteResult = awaitItem()
             while (noteResult.note == null && !noteResult.loading) {
                 noteResult = awaitItem()
             }
@@ -138,13 +166,13 @@ class AdaptiveInteractorTest {
         mainViewModel.onAction(MainAction.OnNoteClick(id))
 
         mainViewModel.stateFlow.test {
-            val selectedResult = awaitItem()
+            val selectedResult: NoteListResult = awaitItem()
             assertTrue(selectedResult is NoteListResult.Success)
             assertEquals(id, selectedResult.selectedId)
 
             noteViewModel.onAction(NoteAction.CheckSaveChange(text))
 
-            val clearedResult = awaitItem()
+            val clearedResult: NoteListResult = awaitItem()
             assertTrue(clearedResult is NoteListResult.Success)
             assertNull(clearedResult.selectedId)
 
@@ -163,4 +191,43 @@ class AdaptiveInteractorTest {
         deferred.await()
         assertEquals(id, adaptiveInteractor.selectedNoteIdStateFlow.value)
     }
+
+    @Test
+    fun `when SettingsCategoriesViewModel selects category then SettingsViewModel reflects selection`() = runTest {
+        settingsCategoriesViewModel.launchCategories()
+        settingsViewModel.launchCollectingSelectedCategoryId()
+
+        settingsViewModel.stateFlow.test {
+            assertNull(awaitItem().selectedCategory)
+
+            settingsCategoriesViewModel.onAction(SettingsCategoriesAction.SelectCategory(SettingsCategory.Security))
+
+            val updated: SecurityResult = awaitItem()
+            assertEquals(SettingsCategory.Security, updated.selectedCategory)
+            cancelAndIgnoreRemainingEvents()
+        }
+        Mockito.verify(mockRouter).adaptiveNavigateToDetail(contentKey = SettingsCategory.Security.ordinal.toLong())
+    }
+
+    @Test
+    fun `when SettingsViewModel navigates back then selection cleared`() = runTest {
+        settingsCategoriesViewModel.launchCategories()
+        settingsViewModel.launchCollectingSelectedCategoryId()
+
+        settingsViewModel.stateFlow.test {
+            assertNull(awaitItem().selectedCategory)
+
+            settingsCategoriesViewModel.onAction(SettingsCategoriesAction.SelectCategory(SettingsCategory.Theme))
+
+            val selected: SecurityResult = awaitItem()
+            assertEquals(SettingsCategory.Theme, selected.selectedCategory)
+
+            settingsViewModel.onAction(SettingsAction.NavBack)
+
+            val cleared: SecurityResult = awaitItem()
+            assertNull(cleared.selectedCategory)
+            cancelAndIgnoreRemainingEvents()
+        }
+        Mockito.verify(mockRouter).adaptiveNavigateBack()
+    }
 }

core/presentation/README.md

@@ -2,6 +2,7 @@ package com.softartdev.notedelight.presentation.settings
 
 import androidx.arch.core.executor.testing.InstantTaskExecutorRule
 import app.cash.turbine.test
+import com.softartdev.notedelight.interactor.AdaptiveInteractor
 import com.softartdev.notedelight.interactor.LocaleInteractor
 import com.softartdev.notedelight.interactor.SnackbarInteractor
 import com.softartdev.notedelight.interactor.SnackbarMessage
@@ -38,7 +39,16 @@ class SettingsViewModelTest {
     private val mockRouter = Mockito.mock(Router::class.java)
     private val mockSnackbarInteractor = Mockito.mock(SnackbarInteractor::class.java)
     private val mockLocaleInteractor = Mockito.mock(LocaleInteractor::class.java)
-    private val settingsViewModel = SettingsViewModel(mockSafeRepo, checkSqlCipherVersionUseCase, mockSnackbarInteractor, mockRouter, RevealFileListUseCase(), mockLocaleInteractor)
+    private val adaptiveInteractor = AdaptiveInteractor()
+    private val settingsViewModel = SettingsViewModel(
+        safeRepo = mockSafeRepo,
+        checkSqlCipherVersionUseCase = checkSqlCipherVersionUseCase,
+        snackbarInteractor = mockSnackbarInteractor,
+        router = mockRouter,
+        revealFileListUseCase = RevealFileListUseCase(),
+        localeInteractor = mockLocaleInteractor,
+        adaptiveInteractor = adaptiveInteractor,
+    )
 
     @After
     fun tearDown() = runTest {
@@ -58,6 +68,23 @@ class SettingsViewModelTest {
     @Test
     fun checkEncryptionFalse() = assertEncryption(false)
 
+    @Test
+    fun refreshUpdatesSwitches() = runTest {
+        Mockito.`when`(mockSafeRepo.databaseState).thenReturn(ENCRYPTED)
+        Mockito.`when`(mockLocaleInteractor.languageEnum).thenReturn(LanguageEnum.ENGLISH)
+        settingsViewModel.stateFlow.test {
+            assertFalse(awaitItem().loading)
+            settingsViewModel.onAction(SettingsAction.Refresh)
+            var result: SecurityResult = awaitItem()
+            while (result.loading) {
+                result = awaitItem()
+            }
+            assertTrue(result.encryption)
+            cancelAndIgnoreRemainingEvents()
+        }
+        Mockito.verifyNoMoreInteractions(mockRouter)
+    }
+
     private fun assertEncryption(encryption: Boolean) = runTest {
         val platformSQLiteState = if (encryption) ENCRYPTED else UNENCRYPTED
         Mockito.`when`(mockSafeRepo.databaseState).thenReturn(platformSQLiteState)