Xargon/ES-DE
1
0
Fork 0
mirror of https://github.com/RetroDECK/ES-DE.git synced 2025-04-10 19:15:13 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fixed an issue where the wrong theme could get selected when deleting the previously used theme via the theme downloader

Browse source
This commit is contained in:
Leon Styhre 2023-08-15 19:54:36 +02:00
parent 4918c16a98
commit e0e370f6b2
1 changed files with 13 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
es-app/src/guis/GuiMenu.cpp
View file

@ -159,7 +159,19 @@ void GuiMenu::openUIOptions()
if (theme->getSelected() != Settings::getInstance()->getString("Theme")) { if (theme->getSelected() != Settings::getInstance()->getString("Theme")) {
Scripting::fireEvent("theme-changed", theme->getSelected(), Scripting::fireEvent("theme-changed", theme->getSelected(),
Settings::getInstance()->getString("Theme")); Settings::getInstance()->getString("Theme"));
Settings::getInstance()->setString("Theme", theme->getSelected()); // Handle the situation where the previously selected theme has been deleted
// using the theme downloader. In this case attempt to fall back to slate-es-de
// and if this theme doesn't exist then select the first available one.
auto themes = ThemeData::getThemes();
if (themes.find(theme->getSelected()) == themes.end()) {
if (themes.find("slate-es-de") != themes.end())
Settings::getInstance()->setString("Theme", "slate-es-de");
else
Settings::getInstance()->setString("Theme", themes.begin()->first);
}
else {
Settings::getInstance()->setString("Theme", theme->getSelected());
}
mWindow->setChangedTheme(); mWindow->setChangedTheme();
// This is required so that the custom collection system does not disappear // This is required so that the custom collection system does not disappear
// if the user is editing a custom collection when switching themes. // if the user is editing a custom collection when switching themes.