Don't throw an exception but use the top as a default value

Author: 0nko

app/src/main/java/com/duckduckgo/app/appearance/AppearanceActivity.kt

@@ -200,9 +200,8 @@ class AppearanceActivity : DuckDuckGoActivity() {
         val subtitle =
             getString(
                 when (omnibarType) {
-                    OmnibarType.SINGLE_TOP -> R.string.settingsAddressBarPositionTop
+                    OmnibarType.SPLIT, OmnibarType.SINGLE_TOP -> R.string.settingsAddressBarPositionTop
                     OmnibarType.SINGLE_BOTTOM -> R.string.settingsAddressBarPositionBottom
-                    OmnibarType.SPLIT -> throw IllegalStateException("Split omnibar not supported in the position selection settings")
                 },
             )
         binding.addressBarPositionSetting.setSecondaryText(subtitle)
@@ -239,9 +238,9 @@ class AppearanceActivity : DuckDuckGoActivity() {
                     override fun onPositiveButtonClicked(selectedItem: Int) {
                         val selectedTheme =
                             when (selectedItem) {
-                                2 -> DuckDuckGoTheme.LIGHT
-                                3 -> DuckDuckGoTheme.DARK
-                                else -> DuckDuckGoTheme.SYSTEM_DEFAULT
+                                2 -> LIGHT
+                                3 -> DARK
+                                else -> SYSTEM_DEFAULT
                             }
                         viewModel.onThemeSelected(selectedTheme)
                     }