From 1e6617d8599057b90b38c54e580db5eb438660f7 Mon Sep 17 00:00:00 2001 From: Helium314 Date: Sat, 8 Feb 2025 21:54:36 +0100 Subject: [PATCH] fix issue with reloading after switching background image --- app/src/main/java/helium314/keyboard/latin/LatinIME.java | 1 + .../main/java/helium314/keyboard/settings/SettingsActivity.kt | 1 - 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/app/src/main/java/helium314/keyboard/latin/LatinIME.java b/app/src/main/java/helium314/keyboard/latin/LatinIME.java index 40e107696..77534fb2e 100644 --- a/app/src/main/java/helium314/keyboard/latin/LatinIME.java +++ b/app/src/main/java/helium314/keyboard/latin/LatinIME.java @@ -1983,6 +1983,7 @@ public class LatinIME extends InputMethodService implements private void reloadIfNecessary() { // better do the reload when showing the keyboard next time, and not on settings change if (SettingsActivityKt.keyboardNeedsReload) { + loadSettings(); KeyboardLayoutSet.onKeyboardThemeChanged(); mKeyboardSwitcher.forceUpdateKeyboardTheme(mDisplayContext); SettingsActivityKt.keyboardNeedsReload = false; diff --git a/app/src/main/java/helium314/keyboard/settings/SettingsActivity.kt b/app/src/main/java/helium314/keyboard/settings/SettingsActivity.kt index 509581957..8e4aea4fb 100644 --- a/app/src/main/java/helium314/keyboard/settings/SettingsActivity.kt +++ b/app/src/main/java/helium314/keyboard/settings/SettingsActivity.kt @@ -16,7 +16,6 @@ import kotlinx.coroutines.flow.MutableStateFlow // todo // calling KeyboardSwitcher.getInstance().forceUpdateKeyboardTheme(requireContext()) while keyboard is showing shows just full screen background // but reload while keyboard is showing would be great (isn't it at least semi-done when changing one-handed mode?) -// bg image inconsistent about being on toolbar or not (is this new?) // performance // find a nice way of testing (probably add logs for measuring time and recompositions) // consider that stuff in composables can get called quite often on any changes