Texmod: keep it a module; give modules a per-frame Draw hook

A plain module only gets Update() (game thread, outside the ImGui frame), so it
can't paint an overlay - which is why Texmod had been promoted to a widget. Add a
no-op ToolboxModule::Draw(IDirect3DDevice9*) called once per render frame for
non-UI modules, so TexmodModule can stay a module and still draw its recording
overlay.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: henderkes

GWToolboxdll/GWToolbox.cpp

@@ -1128,6 +1128,12 @@ void GWToolbox::Draw(IDirect3DDevice9* device)
             }
         }
 
+        // Non-UI modules have no window of their own but may still paint an overlay
+        // this frame (e.g. Texmod's recording banner).
+        for (size_t i = 0; i < other_modules_enabled.size(); i++) {
+            other_modules_enabled[i]->Draw(device);
+        }
+
 #ifdef _DEBUG
         // Feel free to uncomment to play with ImGui's features
         //ImGui::ShowDemoWindow();

GWToolboxdll/Modules/TexmodModule.cpp

@@ -946,13 +946,7 @@ void TexmodModule::Terminate()
 {
     TeardownTextureCapture();
     ShutdownGMod(); // synchronously unloads every pack from gMod
-    ToolboxWidget::Terminate();
-}
-
-void TexmodModule::RegisterSettingsContent()
-{
-    // Plain-module settings: skip ToolboxUIElement's visibility/size/position UI.
-    ToolboxModule::RegisterSettingsContent();
+    ToolboxModule::Terminate();
 }
 
 void TexmodModule::DrawSettingsInternal()

GWToolboxdll/Modules/TexmodModule.h

@@ -1,6 +1,6 @@
 #pragma once
 
-#include <ToolboxWidget.h>
+#include <ToolboxModule.h>
 
 // ---------------------------------------------------------------------------
 // TexmodModule
@@ -10,10 +10,10 @@
 // priority: when two packs replace the same texture, the one higher in the list
 // wins, so reordering re-adds packs in the desired order.
 //
-// A widget (not a plain module) only so it gets a per-frame Draw() for the texture-
-// recording overlay; its settings are still registered like a plain module.
+// Draw() (a plain-module render hook) is used only to paint the texture-recording
+// overlay; the module has no window of its own.
 // ---------------------------------------------------------------------------
-class TexmodModule final : public ToolboxWidget {
+class TexmodModule final : public ToolboxModule {
 public:
     static TexmodModule& Instance()
     {
@@ -35,9 +35,6 @@ class TexmodModule final : public ToolboxWidget {
     void LoadSettings(ToolboxIni* ini) override;
     void SaveSettings(ToolboxIni* ini) override;
 
-    // Register settings like a plain module (no widget visibility/position controls).
-    void RegisterSettingsContent() override;
-
 private:
     TexmodModule() = default;
 };

GWToolboxdll/ToolboxModule.h

@@ -7,6 +7,7 @@
 using SectionDrawCallback = std::function<void(const std::string& section, bool is_showing)>;
 class ToolboxModule;
 class ToolboxIni;
+struct IDirect3DDevice9;
 
 struct SectionDrawCallbackInfo {
     float weighting{};
@@ -72,6 +73,12 @@ class ToolboxModule {
     // Update. Will always be called once every frame. Delta in seconds
     virtual void Update(float) { }
 
+    // Called once per render frame, inside the ImGui frame, for every enabled module.
+    // UI elements override this to draw their window; a plain module can use it to
+    // paint an overlay (e.g. on the background draw list), which it otherwise can't
+    // do from Update() (that runs on the game thread, outside the ImGui frame).
+    virtual void Draw(IDirect3DDevice9*) { }
+
     // This is provided (and called), but use ImGui::GetIO() during update/render if possible.
     virtual bool WndProc(UINT, WPARAM, LPARAM) { return false; }
 

GWToolboxdll/ToolboxUIElement.h

@@ -9,7 +9,7 @@ class ToolboxUIElement : public ToolboxModule {
     [[nodiscard]] bool IsUIElement() const override { return true; }
 
     // Draw user interface. Will be called every frame if the element is visible
-    virtual void Draw(IDirect3DDevice9*) { }
+    void Draw(IDirect3DDevice9*) override { }
 
     void UpdateLocationAgainstSnappedFrame();
     static void UpdateCachedFrameStates();