fix: Provider loader interface being closable and not resizing correctly

lib/libimhex/include/hex/ui/view.hpp

@@ -167,13 +167,12 @@ namespace hex {
 
         void draw() final {
             if (this->shouldDraw()) {
-                ImGui::SetNextWindowSizeConstraints(this->getMinSize(), this->getMaxSize());
-
                 if (this->getWindowOpenState())
                     ImGui::OpenPopup(View::toWindowName(this->getUnlocalizedName()).c_str());
 
                 ImGui::SetNextWindowPos(ImGui::GetMainViewport()->GetCenter(), ImGuiCond_Appearing, ImVec2(0.5F, 0.5F));
-                if (ImGui::BeginPopupModal(View::toWindowName(this->getUnlocalizedName()).c_str(), &this->getWindowOpenState(), ImGuiWindowFlags_NoCollapse | ImGuiWindowFlags_AlwaysAutoResize | this->getWindowFlags())) {
+                ImGui::SetNextWindowSizeConstraints(this->getMinSize(), this->getMaxSize());
+                if (ImGui::BeginPopupModal(View::toWindowName(this->getUnlocalizedName()).c_str(), this->hasCloseButton() ? &this->getWindowOpenState() : nullptr, ImGuiWindowFlags_NoCollapse | ImGuiWindowFlags_AlwaysAutoResize | this->getWindowFlags())) {
                     this->drawContent();
 
                     ImGui::EndPopup();
@@ -183,6 +182,8 @@ namespace hex {
                     this->getWindowOpenState() = false;
             }
         }
+
+        virtual bool hasCloseButton() const { return true; }
     };
 
 }

plugins/builtin/include/content/views/view_provider_settings.hpp

@@ -14,6 +14,13 @@ namespace hex::plugin::builtin {
         [[nodiscard]] bool hasViewMenuItemEntry() const override;
 
         [[nodiscard]] bool shouldDraw() const override { return true; }
+
+        ImVec2 getMinSize() const override { return { -1, -1 }; }
+        ImVec2 getMaxSize() const override { return this->getMinSize(); }
+
+        bool hasCloseButton() const override {
+            return false;
+        }
     };
 
 }

plugins/builtin/romfs/layouts/default.hexlyt

@@ -70,11 +70,6 @@ Pos=1233,38
 Size=687,1021
 Collapsed=0
 
-[Window][###hex.builtin.view.provider_settings.name]
-Pos=822,38
-Size=637,720
-Collapsed=0
-
 [Window][###hex.builtin.view.provider_settings.load_popup]
 Pos=585,322
 Size=314,208

plugins/builtin/romfs/layouts/minimal.hexlyt

@@ -70,11 +70,6 @@ Pos=1233,38
 Size=687,1021
 Collapsed=0
 
-[Window][###hex.builtin.view.provider_settings.name]
-Pos=822,38
-Size=637,720
-Collapsed=0
-
 [Window][###hex.builtin.view.provider_settings.load_popup]
 Pos=585,322
 Size=314,208

plugins/builtin/source/content/recent.cpp

@@ -179,7 +179,7 @@ namespace hex::plugin::builtin::recent {
                     uniqueProviders.insert(RecentEntry {
                         .displayName    = jsonData.at("displayName"),
                         .type           = jsonData.at("type"),
-                        .entryFilePath       = path,
+                        .entryFilePath  = path,
                         .data           = jsonData
                     });
                 } catch (...) { }