Fix quick-keys' spell list's D-pad / controller navigation

Author: Aussiemon

apps/openmw/mwgui/quickkeysmenu.cpp

@@ -228,6 +228,7 @@ namespace MWGui
             mMagicSelectionDialog = std::make_unique<MagicSelectionDialog>(this);
         }
         mMagicSelectionDialog->setVisible(true);
+        mMagicSelectionDialog->setActiveControllerWindow(true);
 
         mAssignDialog->setVisible(false);
     }
@@ -724,7 +725,6 @@ namespace MWGui
         WindowModal::onOpen();
 
         mMagicList->setModel(new SpellModel(MWMechanics::getPlayer()));
-        mMagicList->resetScrollbars();
     }
 
     void MagicSelectionDialog::onModelIndexSelected(SpellModel::ModelIndex index)
@@ -745,4 +745,13 @@ namespace MWGui
 
         return true;
     }
+
+    void MagicSelectionDialog::setActiveControllerWindow(bool active)
+    {
+        if (!Settings::gui().mControllerMenus)
+            return;
+
+        mMagicList->setActiveControllerWindow(active);
+        WindowBase::setActiveControllerWindow(active);
+    }
 }

apps/openmw/mwgui/quickkeysmenu.hpp

@@ -107,6 +107,8 @@ namespace MWGui
         void onOpen() override;
         bool exit() override;
 
+        void setActiveControllerWindow(bool active) override;
+
     private:
         MyGUI::Button* mCancelButton;
         SpellView* mMagicList;
@@ -117,7 +119,6 @@ namespace MWGui
         void onModelIndexSelected(SpellModel::ModelIndex index);
 
         bool onControllerButtonEvent(const SDL_ControllerButtonEvent& arg) override;
-        size_t mControllerFocus = 0;
     };
 }