Skip to content

Commit e316854

Browse files
committed
Fix exit after changing themes.
In caea1c7 , the ThemeChooser's calls to the Reload Theme jump point were modified to include the pop = false argument so that the "old" theme's main menu screen wasn't shown prior to loading the new theme. Unfortunately, this left 2 extra screens on the stack, so we would never again attempt to exit because it looked like we weren't at the top of the main menu. I am working on another approach that will provide behavior closer to that intended in caea1c7, but it's grown large and complicated enough that it will take a while to complete, so I'm committing this change while I finish the other approach. Fixes #9936.
1 parent 8337e87 commit e316854

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

mythtv/programs/mythfrontend/themechooser.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -617,7 +617,7 @@ void ThemeChooser::saveAndReload(MythUIButtonListItem *item)
617617
else
618618
{
619619
gCoreContext->SaveSetting("Theme", info->GetDirectoryName());
620-
GetMythMainWindow()->JumpTo("Reload Theme", false);
620+
GetMythMainWindow()->JumpTo("Reload Theme");
621621
}
622622
}
623623

@@ -797,7 +797,7 @@ void ThemeChooser::customEvent(QEvent *e)
797797
SendMythSystemEvent(event);
798798

799799
gCoreContext->SaveSetting("Theme", m_downloadTheme->GetDirectoryName());
800-
GetMythMainWindow()->JumpTo("Reload Theme", false);
800+
GetMythMainWindow()->JumpTo("Reload Theme");
801801
}
802802
}
803803
}

0 commit comments

Comments
 (0)