Fixed|UI|Client: Loading a savegame from the Home screen

The selected save item got deselected while the UI was being updated during the busy mode, and the save was not loaded.
skyjake · Nov 8, 2018 · 7e35963 · 7e35963
1 parent 4b90b9a
commit 7e35963
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/doomsday/apps/client/src/ui/home/gamepanelbuttonwidget.cpp b/doomsday/apps/client/src/ui/home/gamepanelbuttonwidget.cpp
@@ -212,8 +212,9 @@ DENG_GUI_PIMPL(GamePanelButtonWidget)
             //ClientWindow::main().taskBar().close();
             // TODO: Emit a signal that hides the Home and closes the taskbar.
 
-            // Switch the game.
             gameProfile.setLastPlayedAt();
+
+            // Switch the game.
             DoomsdayApp::app().changeGame(gameProfile, DD_ActivateGameWorker);
 
             if (saves->selectedPos() != ui::Data::InvalidPos)

diff --git a/doomsday/apps/client/src/ui/home/homewidget.cpp b/doomsday/apps/client/src/ui/home/homewidget.cpp
@@ -498,16 +498,16 @@ DENG_GUI_PIMPL(HomeWidget)
         }
 
         // Remove the highlight.
+        const int newHighlightPos = tabs->currentItem().data().toInt();
         for (int pos = 0; pos < columns.size(); ++pos)
         {
-            if (columns[pos]->isHighlighted())
+            if (pos != newHighlightPos && columns[pos]->isHighlighted())
             {
                 columns[pos]->setHighlighted(false);
             }
         }
-
         // Set new highlight.
-        columns[tabs->currentItem().data().toInt()]->setHighlighted(true);
+        columns[newHighlightPos]->setHighlighted(true);
 
         moveShowTimer.start();
     }