From 4de3abdd5a3b2e13af7d7bc5b82f09c06830e32a Mon Sep 17 00:00:00 2001 From: Charles Lombardo Date: Sun, 19 Mar 2023 23:40:15 -0400 Subject: [PATCH] android: Refactor CheckBoxSetting to SwitchSetting --- .../features/settings/model/view/SettingsItem.kt | 2 +- .../view/{CheckBoxSetting.kt => SwitchSetting.kt} | 4 ++-- .../yuzu_emu/features/settings/ui/SettingsAdapter.kt | 4 ++-- .../features/settings/ui/SettingsFragmentPresenter.kt | 10 +++++----- ...SettingViewHolder.kt => SwitchSettingViewHolder.kt} | 8 ++++---- 5 files changed, 14 insertions(+), 14 deletions(-) rename src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/model/view/{CheckBoxSetting.kt => SwitchSetting.kt} (97%) rename src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/ui/viewholder/{CheckBoxSettingViewHolder.kt => SwitchSettingViewHolder.kt} (87%) diff --git a/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/model/view/SettingsItem.kt b/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/model/view/SettingsItem.kt index 91cd1a558..c7ad60046 100644 --- a/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/model/view/SettingsItem.kt +++ b/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/model/view/SettingsItem.kt @@ -23,7 +23,7 @@ abstract class SettingsItem( companion object { const val TYPE_HEADER = 0 - const val TYPE_CHECKBOX = 1 + const val TYPE_SWITCH = 1 const val TYPE_SINGLE_CHOICE = 2 const val TYPE_SLIDER = 3 const val TYPE_SUBMENU = 4 diff --git a/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/model/view/CheckBoxSetting.kt b/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/model/view/SwitchSetting.kt similarity index 97% rename from src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/model/view/CheckBoxSetting.kt rename to src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/model/view/SwitchSetting.kt index c6c6f6f89..b793012cc 100644 --- a/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/model/view/CheckBoxSetting.kt +++ b/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/model/view/SwitchSetting.kt @@ -10,8 +10,8 @@ import org.yuzu.yuzu_emu.features.settings.model.IntSetting import org.yuzu.yuzu_emu.features.settings.model.Setting import org.yuzu.yuzu_emu.features.settings.ui.SettingsFragmentView -class CheckBoxSetting : SettingsItem { - override val type = TYPE_CHECKBOX +class SwitchSetting : SettingsItem { + override val type = TYPE_SWITCH private var defaultValue: Boolean private var showPerformanceWarning: Boolean diff --git a/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/ui/SettingsAdapter.kt b/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/ui/SettingsAdapter.kt index 4eac386cc..1b8a0a5ea 100644 --- a/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/ui/SettingsAdapter.kt +++ b/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/ui/SettingsAdapter.kt @@ -52,7 +52,7 @@ class SettingsAdapter( } SettingsItem.TYPE_CHECKBOX -> { view = inflater.inflate(R.layout.list_item_setting_switch, parent, false) - CheckBoxSettingViewHolder(view, this) + SwitchSettingViewHolder(view, this) } SettingsItem.TYPE_SINGLE_CHOICE, SettingsItem.TYPE_STRING_SINGLE_CHOICE -> { view = inflater.inflate(R.layout.list_item_setting, parent, false) @@ -103,7 +103,7 @@ class SettingsAdapter( notifyDataSetChanged() } - fun onBooleanClick(item: CheckBoxSetting, position: Int, checked: Boolean) { + fun onBooleanClick(item: SwitchSetting, position: Int, checked: Boolean) { val setting = item.setChecked(checked) if (setting != null) { fragmentView.putSetting(setting) diff --git a/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/ui/SettingsFragmentPresenter.kt b/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/ui/SettingsFragmentPresenter.kt index f475b26c4..ef825ec58 100644 --- a/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/ui/SettingsFragmentPresenter.kt +++ b/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/ui/SettingsFragmentPresenter.kt @@ -139,7 +139,7 @@ class SettingsFragmentPresenter(private val fragmentView: SettingsFragmentView) val cpuAccuracy = cpuSection!!.getSetting(SettingsFile.KEY_CPU_ACCURACY) sl.apply { add( - CheckBoxSetting( + SwitchSetting( SettingsFile.KEY_RENDERER_USE_SPEED_LIMIT, Settings.SECTION_RENDERER, frameLimitEnable, @@ -184,7 +184,7 @@ class SettingsFragmentPresenter(private val fragmentView: SettingsFragmentView) val language = systemSection.getSetting(SettingsFile.KEY_LANGUAGE_INDEX) sl.apply { add( - CheckBoxSetting( + SwitchSetting( SettingsFile.KEY_USE_DOCKED_MODE, Settings.SECTION_SYSTEM, dockedMode, @@ -283,7 +283,7 @@ class SettingsFragmentPresenter(private val fragmentView: SettingsFragmentView) ) ) add( - CheckBoxSetting( + SwitchSetting( SettingsFile.KEY_RENDERER_FORCE_MAX_CLOCK, Settings.SECTION_RENDERER, rendererForceMaxClocks, @@ -293,7 +293,7 @@ class SettingsFragmentPresenter(private val fragmentView: SettingsFragmentView) ) ) add( - CheckBoxSetting( + SwitchSetting( SettingsFile.KEY_RENDERER_ASYNCHRONOUS_SHADERS, Settings.SECTION_RENDERER, rendererAsynchronousShaders, @@ -303,7 +303,7 @@ class SettingsFragmentPresenter(private val fragmentView: SettingsFragmentView) ) ) add( - CheckBoxSetting( + SwitchSetting( SettingsFile.KEY_RENDERER_DEBUG, Settings.SECTION_RENDERER, rendererDebug, diff --git a/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/ui/viewholder/CheckBoxSettingViewHolder.kt b/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/ui/viewholder/SwitchSettingViewHolder.kt similarity index 87% rename from src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/ui/viewholder/CheckBoxSettingViewHolder.kt rename to src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/ui/viewholder/SwitchSettingViewHolder.kt index 3296aed68..f8b229830 100644 --- a/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/ui/viewholder/CheckBoxSettingViewHolder.kt +++ b/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/settings/ui/viewholder/SwitchSettingViewHolder.kt @@ -8,13 +8,13 @@ import android.widget.CompoundButton import android.widget.TextView import com.google.android.material.materialswitch.MaterialSwitch import org.yuzu.yuzu_emu.R -import org.yuzu.yuzu_emu.features.settings.model.view.CheckBoxSetting +import org.yuzu.yuzu_emu.features.settings.model.view.SwitchSetting import org.yuzu.yuzu_emu.features.settings.model.view.SettingsItem import org.yuzu.yuzu_emu.features.settings.ui.SettingsAdapter -class CheckBoxSettingViewHolder(itemView: View, adapter: SettingsAdapter) : +class SwitchSettingViewHolder(itemView: View, adapter: SettingsAdapter) : SettingViewHolder(itemView, adapter) { - private lateinit var item: CheckBoxSetting + private lateinit var item: SwitchSetting private lateinit var textSettingName: TextView private lateinit var textSettingDescription: TextView private lateinit var switch: MaterialSwitch @@ -26,7 +26,7 @@ class CheckBoxSettingViewHolder(itemView: View, adapter: SettingsAdapter) : } override fun bind(item: SettingsItem) { - this.item = item as CheckBoxSetting + this.item = item as SwitchSetting textSettingName.setText(item.nameId) if (item.descriptionId!! > 0) { textSettingDescription.setText(item.descriptionId)