Add Drawlist Vertex and Indexcount geteter (for debugging usage only)

Throw OneFrame living objs into a pool

backends/source/gfx_directx9.cpp

@@ -175,6 +175,10 @@ void GfxDirectX9::SysReset() {
   impl->Device->SetRenderState(D3DRS_ALPHABLENDENABLE, TRUE);
   impl->Device->SetRenderState(D3DRS_SRCBLEND, D3DBLEND_SRCALPHA);
   impl->Device->SetRenderState(D3DRS_DESTBLEND, D3DBLEND_INVSRCALPHA);
+
+  impl->Device->SetSamplerState(0, D3DSAMP_MINFILTER, D3DTEXF_LINEAR);
+  impl->Device->SetSamplerState(0, D3DSAMP_MAGFILTER, D3DTEXF_LINEAR);
+  impl->Device->SetSamplerState(0, D3DSAMP_MIPFILTER, D3DTEXF_LINEAR);
 }
 
 Li::Texture GfxDirectX9::LoadTexture(const std::vector<PD::u8>& pixels, int w,
@@ -257,27 +261,27 @@ void GfxDirectX9::UploadPools() {
   if (!impl->VBO || impl->VertexBufferSize != GetVertexPoolSize()) {
     if (impl->VBO) impl->VBO->Release();
     impl->Device->CreateVertexBuffer(GetVertexPoolSize() * sizeof(Li::Vertex),
-                                     D3DUSAGE_DYNAMIC | D3DUSAGE_WRITEONLY, 0,
+                                     D3DUSAGE_WRITEONLY, 0, D3DPOOL_MANAGED,
-                                     D3DPOOL_DEFAULT, &impl->VBO, nullptr);
+                                     &impl->VBO, nullptr);
     impl->VertexBufferSize = GetVertexPoolSize();
   }
 
   if (!impl->IBO || impl->IndexBufferSize != GetIndexPoolSize()) {
     if (impl->IBO) impl->IBO->Release();
-    impl->Device->CreateIndexBuffer(
+    impl->Device->CreateIndexBuffer(GetIndexPoolSize() * sizeof(u16),
-        GetIndexPoolSize() * sizeof(u16), D3DUSAGE_DYNAMIC | D3DUSAGE_WRITEONLY,
+                                    D3DUSAGE_WRITEONLY, D3DFMT_INDEX16,
-        D3DFMT_INDEX16, D3DPOOL_DEFAULT, &impl->IBO, nullptr);
+                                    D3DPOOL_MANAGED, &impl->IBO, nullptr);
     impl->IndexBufferSize = GetIndexPoolSize();
   }
 
   void* vptr;
-  impl->VBO->Lock(0, 0, &vptr, D3DLOCK_DISCARD);
+  impl->VBO->Lock(0, 0, &vptr, 0);
   memcpy(vptr, GetVertexBufPtr(0),
          GetVertexPoolSize() * sizeof(PD::Li::Vertex));
   impl->VBO->Unlock();
 
   void* iptr;
-  impl->IBO->Lock(0, 0, &iptr, D3DLOCK_DISCARD);
+  impl->IBO->Lock(0, 0, &iptr, 0);
   memcpy(iptr, GetIndexBufPtr(0), GetIndexPoolSize() * sizeof(u16));
   impl->IBO->Unlock();
 }

include/pd/drivers/gfx.hpp

@@ -125,7 +125,8 @@ class GfxDriverBase : public GfxDriver {
       while (index < commands.size() &&
              CurrentTexIsSDF == commands[index].SDF &&
              (CurrentTex == commands[index].Tex ||
-              (CurrentTex == pWhite.GetID() && commands[index].Tex == 0))) {
+              (CurrentTex == pWhite.GetID() && commands[index].Tex == 0)) &&
+             commands[index].FirstIndex == startidx + num_indices) {
         num_indices += commands[index].IndexCount;
         index++;
       }

include/pd/lithium/drawlist.hpp

@@ -43,6 +43,18 @@ class PD_API Drawlist {
   void Optimize();
   void Clear();
 
+  size_t GetNumVertices() const {
+    size_t count = 0;
+    for (const auto& cmd : pCommands) count += cmd.VertexCount;
+    return count;
+  }
+
+  size_t GetNumIndices() const {
+    size_t count = 0;
+    for (const auto& cmd : pCommands) count += cmd.IndexCount;
+    return count;
+  }
+
   /** Command Allocation */
   Command& NewCommand();
   bool HasCommands() { return pCommands.size() > 0; }

include/pd/ui7/container/coloredit.hpp

@@ -47,7 +47,9 @@ class PD_API ColorEdit : public Container {
     this->initial_color = *color;
     this->tdim = io.Font->GetTextBounds(label.c_str(), io.FontScale);
   }
-  ~ColorEdit() = default;
+  ~ColorEdit() {
+    if (layout) delete layout;
+  }
 
   /**
    * Override for the Input Handler
@@ -68,7 +70,7 @@ class PD_API ColorEdit : public Container {
   u32* color_ref = nullptr;  ///< Color Reference
   u32 initial_color;         ///< Initial Color
   std::string label;         ///< Label of the Button
-  Layout* layout;            ///< Layout to open
+  Layout* layout = nullptr;  ///< Layout to open
   bool is_shown = false;     ///< AHow Layout Editor
 };
 }  // namespace UI7

include/pd/ui7/container/container.hpp

@@ -111,6 +111,25 @@ class PD_API Container {
   virtual void Draw() {}
   /** Template function to update internal data (if needed) */
   virtual void Update() {}
+  /** Template as well as base func for Pool based Containers */
+  virtual void Reset() {
+    pos = 0;
+    size = 0;
+    parent = nullptr;
+    id = 0;
+    pFlags = 0;
+    skippable = false;
+    rem = false;
+    inp_done = false;
+    pSelected = false;
+    pPressed = false;
+    pPressedTwice = false;
+    pCLipRectUsed = false;
+    pClipRect = 0;
+    io = nullptr;
+    list = nullptr;
+    last_use = 0;
+  }
 
   /** Internal function */
   void PreDraw();
@@ -155,11 +174,11 @@ class PD_API Container {
   /** Container Size*/
   fvec2 size;
   /** Reference to the Drawlist to Draw to*/
-  Li::Drawlist* list;
+  Li::Drawlist* list = nullptr;
   /** IO Reference for Renderer and Theme */
-  UI7::IO* io;
+  UI7::IO* io = nullptr;
   /** Reference to the parent container*/
-  Container* parent;
+  Container* parent = nullptr;
   /** Object ID (0 if unused)*/
   u32 id = 0;
   /** Internal Flags */

include/pd/ui7/container/dynobj.hpp

@@ -23,9 +23,9 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 SOFTWARE.
 */
 
+#include <functional>
 #include <pd/ui7/container/container.hpp>
 #include <pd/ui7/io.hpp>
-#include <functional>
 
 namespace PD {
 namespace UI7 {
@@ -38,6 +38,7 @@ namespace UI7 {
  */
 class PD_API DynObj : public Container {
  public:
+  DynObj() {}
   /**
    * Button Object constructor
    * @param label Label of the Button
@@ -69,6 +70,14 @@ class PD_API DynObj : public Container {
   /** Function to Update Size if framepadding changes */
   void Update() override;
 
+  void Reset() override {
+    Container::Reset();
+    color = UI7Color_Button;
+    pressed = false;
+    pRenFun = nullptr;
+    pInp = nullptr;
+  }
+
  private:
   UI7Color color = UI7Color_Button;  ///< current button color
   bool pressed = false;              ///< ispressed value

include/pd/ui7/container/image.hpp

@@ -32,6 +32,7 @@ namespace UI7 {
  */
 class PD_API Image : public Container {
  public:
+  Image() {}
   /**
    * Constructor for the Image Object
    * @param img Image Texture Reference
@@ -57,6 +58,13 @@ class PD_API Image : public Container {
    * */
   void Draw() override;
 
+  void Reset() override {
+    Container::Reset();
+    img = Li::Texture();
+    newsize = 0.f;
+    cuv = Li::Rect();
+  }
+
  private:
   Li::Texture img;      ///< Texture
   fvec2 newsize = 0.f;  ///< New Size

include/pd/ui7/container/label.hpp

@@ -32,6 +32,7 @@ namespace UI7 {
  */
 class PD_API Label : public Container {
  public:
+  Label() {}
   /**
    * Constructor for Label Object
    * @param label Label [Text] to Draw
@@ -54,6 +55,13 @@ class PD_API Label : public Container {
    */
   void Update() override;
 
+  void Reset() override {
+    Container::Reset();
+    tdim = 0;
+    color = UI7Color_Text;
+    label.clear();
+  }
+
  private:
   fvec2 tdim;                      ///< Text Size
   UI7Color color = UI7Color_Text;  ///< Color

include/pd/ui7/io.hpp

@@ -31,16 +31,14 @@ SOFTWARE.
 #include <pd/ui7/viewport.hpp>
 
 namespace PD {
-class Context;
 namespace UI7 {
+class Label;
+class Image;
+class DynObj;
 class PD_API IO {
  public:
-  IO() : DeltaStats(60), CurrentViewPort("", 0) {
+  IO();
-    /** Probably not the best solution i guess */
+  ~IO();
-    // CurrentViewPort =
-    //   ViewPort::New("Default", ivec4(ivec2(0, 0), pCtx.Gfx()->ViewPort));
-  }
-  ~IO() {}
 
   /**
    * IO Update Internal Variables
@@ -84,6 +82,11 @@ class PD_API IO {
   u32 NumIndices = 0;   ///< Debug Indices Num
   std::vector<u32> MenuOrder;
 
+  // Pools
+  PD::Pool<UI7::Label> LabelPool;
+  PD::Pool<UI7::Image> ImagePool;
+  PD::Pool<UI7::DynObj> DynObjPool;
+
   // DrawlistApi
   void RegisterDrawlist(const UI7::ID& id, Li::Drawlist* v) {
     DrawlistRegestry.push_back(std::make_pair(id, v));

include/pd/ui7/layout.hpp

@@ -47,7 +47,7 @@ class PD_API Layout {
     Size = 0;
     WorkRect = fvec4(IO.MenuPadding, Size - (fvec2(2) * IO.MenuPadding));
   }
-  ~Layout() = default;
+  ~Layout();
 
   /** SECTION CONTAINERS */
   /**

include/pd/ui7/ui7.hpp

@@ -56,8 +56,7 @@ class PD_API Context {
   void AddViewPort(const ID& id, const ivec4& vp);
   void UseViewPort(const ID& id);
   void Update();
-  Menu* BeginMenu(const ID& id, UI7MenuFlags flags = 0,
+  Menu* BeginMenu(const ID& id, UI7MenuFlags flags = 0, bool* pShow = nullptr);
-                      bool* pShow = nullptr);
   Menu* CurrentMenu() { return pCurrent; }
   void EndMenu();
   void AboutMenu(bool* show = nullptr);

source/ui7/container/coloredit.cpp

@@ -33,8 +33,7 @@ PD_API void ColorEdit::HandleInput() {
   }
   // Assert(screen.get(), "Screen is not set up!");
   // if (screen->ScreenType() == Screen::Bottom) {
-  if (io->InputHandler.DragObject(this->GetID() + 2,
+  if (io->InputHandler.DragObject(this->GetID() + 2, fvec4(FinalPos(), size))) {
-                                   fvec4(FinalPos(), size))) {
     if (io->InputHandler.DragReleasedAW) {
       is_shown = !is_shown;
     }
@@ -54,18 +53,18 @@ PD_API void ColorEdit::Draw() {
       layout = new UI7::Layout(GetID(), *io);
     }
     layout->SetPosition(FinalPos());
-    layout->AddObjectEx(
+    DynObj* r = io->DynObjPool.Allocate();
-        new DynObj(
+    *r = UI7::DynObj([=, this](UI7::IO* io, Li::Drawlist* l, Container* thiz) {
-            [=, this](UI7::IO* io, Li::Drawlist* l, Container* thiz) {
       thiz->SetSize(layout->GetSize());
       // l->Layer(30);
       l->PathRect(thiz->GetPos(), thiz->GetPos() + thiz->GetSize(),
                   io->FrameRounding);
       l->PathFill(io->Theme.Get(UI7Color_FrameBackground));
-            }),
+    });
-        UI7LytAdd_Front | UI7LytAdd_NoCursorUpdate | UI7LytAdd_NoScrollHandle);
+    layout->AddObjectEx(r, UI7LytAdd_Front | UI7LytAdd_NoCursorUpdate |
-    auto obj = new DynObj(
+                               UI7LytAdd_NoScrollHandle);
-        [=, this](UI7::IO* io, Li::Drawlist* l, Container* thiz) {
+    r = io->DynObjPool.Allocate();
+    *r = UI7::DynObj([=, this](UI7::IO* io, Li::Drawlist* l, Container* thiz) {
       l->PathRect(thiz->FinalPos(), thiz->FinalPos() + io->ItemRowHeight,
                   io->FrameRounding);
       l->PathFill(*color_ref);
@@ -73,8 +72,8 @@ PD_API void ColorEdit::Draw() {
           thiz->FinalPos() + fvec2(io->ItemSpace.x + io->ItemRowHeight, 0),
           label.c_str(), io->Theme.Get(UI7Color_Text));
     });
-    obj->SetSize(PD::fvec2(200, io->ItemRowHeight));
+    r->SetSize(PD::fvec2(200, io->ItemRowHeight));
-    layout->AddObject(obj);
+    layout->AddObject(r);
     layout->Label("RGBA: ({}, {}, {}, {})", *((u8*)color_ref),
                   *(((u8*)color_ref) + 1), *(((u8*)color_ref) + 2),
                   *(((u8*)color_ref) + 3));

source/ui7/container/container.cpp

@@ -31,8 +31,8 @@ PD_API void Container::HandleScrolling(fvec2 scrolling, fvec4 viewport) {
   }
   last_use = PD::Os::GetTime();
   pos -= fvec2(0, scrolling.y);
-  skippable = !Li::Math::InBounds(
+  skippable =
-      pos, size,
+      !Li::Math::InBounds(pos, size,
                           fvec4(viewport.x, viewport.y, viewport.x + viewport.z,
                                 viewport.y + viewport.w));
 }

source/ui7/container/dragdata.cpp

@@ -21,6 +21,7 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 SOFTWARE.
  */
 
+#include <algorithm>
 #include <pd/ui7/container/dragdata.hpp>
 #include <pd/ui7/container/label.hpp>
 #include <type_traits>
@@ -92,8 +93,8 @@ PD_API void DragData<T>::Draw() {
     // list->LayerDown();
     off_x += td.x + io->ItemSpace.x + io->FramePadding.x;
   }
-  list->DrawText(FinalPos() + fvec2(off_x, io->FramePadding.y * 0.5), label.c_str(),
+  list->DrawText(FinalPos() + fvec2(off_x, io->FramePadding.y * 0.5),
-                 io->Theme.Get(UI7Color_Text));
+                 label.c_str(), io->Theme.Get(UI7Color_Text));
 }
 
 template <typename T>

source/ui7/container/slider.cpp

@@ -21,6 +21,7 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 SOFTWARE.
  */
 
+#include <algorithm>
 #include <pd/ui7/container/label.hpp>
 #include <pd/ui7/container/slider.hpp>
 #include <type_traits>
@@ -56,8 +57,8 @@ PD_API void Slider<T>::HandleInput() {
           fvec4(FinalPos() + fvec2(2, 0), fvec2(width, GetSize().y)))) {
     if (!io->InputHandler.DragReleasedAW) {
       *data = std::clamp(
-          T(max * (std::clamp(io->InputHandler.DragLastPosition.x - xps, 0.f,
+          T(max *
-                              width) /
+            (std::clamp(io->InputHandler.DragLastPosition.x - xps, 0.f, width) /
              width)),
           this->min, this->max);
     }

source/ui7/io.cpp

@@ -22,10 +22,23 @@ SOFTWARE.
  */
 
 #include <pd/core/core.hpp>
-#include <pd/ui7/io.hpp>
 #include <pd/drivers/drivers.hpp>
+#include <pd/ui7/containers.hpp>
+#include <pd/ui7/io.hpp>
 
 namespace PD {
+
+PD_API UI7::IO::IO() : DeltaStats(60), CurrentViewPort("", 0) {
+  /** Probably not the best solution i guess */
+  // CurrentViewPort =
+  //   ViewPort::New("Default", ivec4(ivec2(0, 0), pCtx.Gfx()->ViewPort));
+  // Start a little larger on these
+  LabelPool.Init(512);
+  DynObjPool.Init(512);
+}
+
+PD_API UI7::IO::~IO() {}
+
 PD_API void UI7::IO::Update() {
   /** Todo: find out if we even still use the Drawlist regestry */
   u64 current = PD::Os::GetTimeNano();
@@ -39,7 +52,10 @@ PD_API void UI7::IO::Update() {
   DrawlistRegestry.push_front(std::make_pair("CtxBackList", &Back));
   if (Font) ItemRowHeight = FontScale * Font->PixelHeight;
   // RegisterDrawList("CtxBackList", Back);
-  NumIndices = 0;//FDL.pNumIndices;
+  NumIndices = FDL.GetNumIndices();
-  NumVertices = 0;//FDL.pNumVertices;
+  NumVertices = FDL.GetNumVertices();
+  LabelPool.ResetFast();
+  DynObjPool.ResetFast();
+  ImagePool.ResetFast();
 }
 }  // namespace PD

source/ui7/layout.cpp

@@ -26,6 +26,15 @@ SOFTWARE.
 
 namespace PD {
 namespace UI7 {
+
+PD_API Layout::~Layout() {
+  // We all love managing memory i guess
+  for (Container* obj : IDObjects) {
+    delete obj;
+  }
+  IDObjects.clear();
+}
+
 PD_API void Layout::CursorInit() { Cursor = fvec2(WorkRect.x, WorkRect.y); }
 
 PD_API void Layout::SameLine() {
@@ -136,6 +145,7 @@ PD_API void Layout::Update() {
 
   for (auto it = IDObjects.begin(); it != IDObjects.end();) {
     if ((*it)->Removable()) {
+      delete *it;
       it = IDObjects.erase(it);
     } else {
       it++;
@@ -151,7 +161,8 @@ PD_API void Layout::Update() {
 
 PD_API void Layout::Label(const std::string& label) {
   // Layout API
-  auto r = new UI7::Label(label, IO);
+  auto r = IO.LabelPool.Allocate();
+  *r = UI7::Label(label, IO);
   r->SetClipRect(fvec4(GetPosition(), GetPosition() + GetSize()));
   AddObject(r);
 }
@@ -182,7 +193,8 @@ PD_API void Layout::Checkbox(const std::string& label, bool& v) {
 }
 
 PD_API void Layout::Image(Li::Texture img, fvec2 size, Li::Rect uv) {
-  Container* r = new UI7::Image(img, size, uv);
+  auto r = IO.ImagePool.Allocate();
+  *r = UI7::Image(img, size, uv);
   AddObject(r);
 }
 

source/ui7/menu.cpp

@@ -21,6 +21,7 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 SOFTWARE.
 */
 
+#include <algorithm>
 #include <pd/ui7/containers.hpp>
 #include <pd/ui7/menu.hpp>
 
@@ -35,7 +36,8 @@ Menu::Menu(const ID& id, IO& io) : pIO(io), pID(id), pLayout(id, io) {
 
 PD_API void Menu::Label(const std::string& label) {
   // Layout API
-  auto r = new UI7::Label(label, pIO);
+  auto r = pIO.LabelPool.Allocate();
+  *r = UI7::Label(label, pIO);
   pLayout.AddObject(r);
 }
 
@@ -67,7 +69,8 @@ PD_API void Menu::Checkbox(const std::string& label, bool& v) {
 }
 
 PD_API void Menu::Image(Li::Texture img, fvec2 size, Li::Rect uv) {
-  Container* r = new UI7::Image(img, size, uv);
+  auto r = pIO.ImagePool.Allocate();
+  *r = UI7::Image(img, size, uv);
   pLayout.AddObject(r);
 }
 
@@ -83,7 +86,8 @@ PD_API void Menu::ColorEdit(const std::string& label, u32& clr) {
 
 PD_API void Menu::Separator() {
   // Dynamic Objects are very simple...
-  Container* r = new DynObj(
+  DynObj* r = pIO.DynObjPool.Allocate();
+  *r = UI7::DynObj(
       [=, this](UI7::IO* io, Li::Drawlist* l, UI7::Container* self) {
         l->DrawRectFilled(self->FinalPos(), self->GetSize(),
                           pIO.Theme.Get(UI7Color_TextDead));
@@ -97,7 +101,8 @@ PD_API void Menu::Separator() {
 
 PD_API void Menu::SeparatorText(const std::string& label) {
   // Also note to use [=] instead of [&] to not undefined access label
-  Container* r = new DynObj([=, this](UI7::IO* io, Li::Drawlist* l,
+  DynObj* r = pIO.DynObjPool.Allocate();
+  *r = UI7::DynObj([=, this](UI7::IO* io, Li::Drawlist* l,
                              UI7::Container* self) {
     fvec2 size = self->GetSize();
     fvec2 tdim = io->Font->GetTextBounds(label.c_str(), io->FontScale);
@@ -250,8 +255,8 @@ PD_API void Menu::DrawBaseLayout() {
   if (pIsOpen) {
     /** Resize Sym (Render on Top of Everything) */
     if (!(Flags & UI7MenuFlags_NoResize)) {
-      Container* r =
+      DynObj* r = pIO.DynObjPool.Allocate();
-          new DynObj([](IO* io, Li::Drawlist* l, UI7::Container* self) {
+      *r = UI7::DynObj([](IO* io, Li::Drawlist* l, UI7::Container* self) {
         // //l->Layer(1);
         l->PathAdd(self->FinalPos() + self->GetSize() - fvec2(0, 20));
         l->PathAdd(self->FinalPos() + self->GetSize());
@@ -266,8 +271,8 @@ PD_API void Menu::DrawBaseLayout() {
     }
 
     /** Background */
-    Container* r = new DynObj([](IO* io, Li::Drawlist* l,
+    DynObj* r = pIO.DynObjPool.Allocate();
-                                  UI7::Container* self) {
+    *r = UI7::DynObj([](IO* io, Li::Drawlist* l, UI7::Container* self) {
       // l->Layer(0);
       l->PathRectEx(self->FinalPos(), self->FinalPos() + self->GetSize(), 10.f,
                     LiPathRectFlags_KeepTop | LiPathRectFlags_KeepBot);
@@ -283,7 +288,8 @@ PD_API void Menu::DrawBaseLayout() {
                                UI7LytAdd_Front);
   }
   if (!(Flags & UI7MenuFlags_NoTitlebar)) {
-    Container* r = new  DynObj(
+    DynObj* r = pIO.DynObjPool.Allocate();
+    *r = UI7::DynObj(
         [=, this](UI7::IO* io, Li::Drawlist* l, UI7::Container* self) {
           // l->Layer(20);
           /** Header Bar */
@@ -306,7 +312,8 @@ PD_API void Menu::DrawBaseLayout() {
 
     /** Collapse Sym */
     if (!(Flags & UI7MenuFlags_NoCollapse)) {
-      r = new DynObj([=, this](UI7::IO* io, Li::Drawlist* l,
+      DynObj* r = pIO.DynObjPool.Allocate();
+      *r = UI7::DynObj([=, this](UI7::IO* io, Li::Drawlist* l,
                                  UI7::Container* self) {
         /** This sym actually requires layer 21 (i dont know why) */
         // l->Layer(21);
@@ -382,7 +389,8 @@ PD_API bool Menu::BeginTreeNode(const ID& id) {
   }
 
   // Object
-  auto r = new DynObj([=, this](IO* io, Li::Drawlist* l, Container* self) {
+  DynObj* r = pIO.DynObjPool.Allocate();
+  *r = UI7::DynObj([=, this](IO* io, Li::Drawlist* l, Container* self) {
     fvec2 ts = self->FinalPos() + fvec2(0, 7);
     fvec2 pl[2] = {fvec2(10, 5), fvec2(0, 10)};
     if (n->second) {
@@ -393,13 +401,12 @@ PD_API bool Menu::BeginTreeNode(const ID& id) {
     l->DrawTriangleFilled(ts, ts + pl[0], ts + pl[1],
                           io->Theme.Get(UI7Color_FrameBackground));
 
-    l->DrawText(self->FinalPos() + fvec2(10 + io->ItemSpace.x, 0), id.GetName().c_str(),
+    l->DrawText(self->FinalPos() + fvec2(10 + io->ItemSpace.x, 0),
-                io->Theme.Get(UI7Color_Text));
+                id.GetName().c_str(), io->Theme.Get(UI7Color_Text));
   });
   /** Yes this new function handler was created for tree nodes */
   r->AddInputHandler([=, this](IO* io, Container* self) {
-    if (io->InputHandler.DragObject(
+    if (io->InputHandler.DragObject(ID(pID.GetName() + id.GetName()),
-            ID(pID.GetName() + id.GetName()),
                                     fvec4(self->FinalPos(), self->GetSize()))) {
       if (io->InputHandler.DragReleased) {
         n->second = !n->second;

source/ui7/ui7.cpp

@@ -51,8 +51,7 @@ PD_API void Context::UseViewPort(const ID& id) {
   pIO.CurrentViewPort = pIO.ViewPorts[id.RawID()];
 }
 
-PD_API Menu* Context::BeginMenu(const ID& id, UI7MenuFlags flags,
+PD_API Menu* Context::BeginMenu(const ID& id, UI7MenuFlags flags, bool* pShow) {
-                                    bool* pShow) {
   if (pCurrent) {
     PDERR("UI7: You are already in {} Menu!", pCurrent->pID.GetName());
     return nullptr;
@@ -256,8 +255,7 @@ PD_API void Context::MetricsMenu(bool* show) {
           m->Label(std::format("Cursor: {}", pMenus[it]->pLayout.Cursor));
           if (m->BeginTreeNode(
                   "ID Objects (" +
-                  std::to_string(pMenus[it]->pLayout.IDObjects.size()) +
+                  std::to_string(pMenus[it]->pLayout.IDObjects.size()) + ")")) {
-                  ")")) {
             for (auto& jt : pMenus[it]->pLayout.IDObjects) {
               m->Label(std::format("{:08X}", jt->GetID()));
             }

tests/gfx/source/main.cpp

@@ -303,6 +303,16 @@ int main(int argc, char** argv) {
       m->Label("Hello World!");
       ui7.EndMenu();
     }
+    if (auto m = ui7.BeginMenu("LI DBG INFO")) {
+      m->Label("Gfx Driver: {}", PD::Gfx::GetDriverName());
+      m->Separator();
+      m->Label("LI  Draw Comamnds: {}", PD::Gfx::GetNumCommands());
+      m->Label("GFX Draw Calls: {}", PD::Gfx::GetNumDrawcalls());
+      m->Label("GFX Vertices: {}", PD::Gfx::GetNumVertices());
+      m->Label("GFX Triangles: {}", PD::Gfx::GetNumVertices() / 3);
+      m->Label("GFX Indices: {}", PD::Gfx::GetNumIndices());
+      ui7.EndMenu();
+    }
     ui7.MetricsMenu();
     PD::TT::End("BuildUI7Menus");
     PD::TT::Beg("UI7::Context::Update");
@@ -320,6 +330,7 @@ int main(int argc, char** argv) {
     PD::TT::Beg("OSCTX::MainLoop");
   }
   font.Delete();
+  debug_font.Delete();
   PD::Gfx::DeleteTexture(pTex);
   PD::Gfx::Deinit();
   pOs->Deinit();

tests/gfx/source/os/desktopos.cpp

@@ -18,6 +18,7 @@ struct DesktopOS::Impl {
 #if WIN32
   IDirect3D9* d3d = nullptr;
   IDirect3DDevice9* dx9_device = nullptr;
+  D3DPRESENT_PARAMETERS d3dpp = {};
 #endif
 };
 
@@ -56,17 +57,18 @@ void DesktopOS::Init() {
   if (pDriver == Driver::DirectX9) {
     impl->d3d = Direct3DCreate9(D3D_SDK_VERSION);
     auto hwnd = glfwGetWin32Window(impl->win);
-    D3DPRESENT_PARAMETERS d3dpp = {};
+
-    d3dpp.Windowed = TRUE;
+    impl->d3dpp = D3DPRESENT_PARAMETERS{};
-    d3dpp.SwapEffect = D3DSWAPEFFECT_DISCARD;
+    impl->d3dpp.Windowed = TRUE;
-    d3dpp.hDeviceWindow = hwnd;
+    impl->d3dpp.SwapEffect = D3DSWAPEFFECT_DISCARD;
-    d3dpp.BackBufferFormat = D3DFMT_UNKNOWN;
+    impl->d3dpp.hDeviceWindow = hwnd;
-    d3dpp.EnableAutoDepthStencil = TRUE;
+    impl->d3dpp.BackBufferFormat = D3DFMT_UNKNOWN;
-    d3dpp.AutoDepthStencilFormat = D3DFMT_D16;
+    impl->d3dpp.EnableAutoDepthStencil = TRUE;
+    impl->d3dpp.AutoDepthStencilFormat = D3DFMT_D16;
 
     HRESULT hr = impl->d3d->CreateDevice(
         D3DADAPTER_DEFAULT, D3DDEVTYPE_HAL, hwnd,
-        D3DCREATE_HARDWARE_VERTEXPROCESSING, &d3dpp, &impl->dx9_device);
+        D3DCREATE_HARDWARE_VERTEXPROCESSING, &impl->d3dpp, &impl->dx9_device);
     if (FAILED(hr)) {
       MessageBoxW(nullptr, L"Failed to create D3D9 device", L"Error", MB_OK);
       std::abort();
@@ -100,7 +102,15 @@ void DesktopOS::ClearViewPort() {
     glViewport(0, 0, pViewPort.x, pViewPort.y);
   } else if (pDriver == Driver::DirectX9) {
 #ifdef _WIN32
-    if (impl->dx9_device) {
+    // Resize the swapchain to match the window size
+    if (impl->dx9_device && pViewPort.x > 0 && pViewPort.y > 0) {
+      if (impl->d3dpp.BackBufferWidth != (UINT)pViewPort.x ||
+          impl->d3dpp.BackBufferHeight != (UINT)pViewPort.y) {
+        impl->d3dpp.BackBufferWidth = pViewPort.x;
+        impl->d3dpp.BackBufferHeight = pViewPort.y;
+        impl->dx9_device->Reset(&impl->d3dpp);
+      }
+
       impl->dx9_device->Clear(0, nullptr, D3DCLEAR_TARGET | D3DCLEAR_ZBUFFER,
                               D3DCOLOR_XRGB(25, 25, 25), 1.0f, 0);
       impl->dx9_device->BeginScene();
@@ -112,7 +122,8 @@ void DesktopOS::ClearViewPort() {
 void DesktopOS::SwapBuffers() {
   if (pDriver == Driver::DirectX9) {
 #ifdef _WIN32
-    if (impl->dx9_device) {
+    // yes we schould use some safetey checks
+    if (impl->dx9_device && pViewPort.x > 0 && pViewPort.y > 0) {
       impl->dx9_device->EndScene();
       impl->dx9_device->Present(nullptr, nullptr, nullptr, nullptr);
     }