commit 8b7668d24f8b2899b5df4f1593b7bee21425f3ff
author ocornut <omarcornut@gmail.com> Thu Mar 26 16:03:30 2015 +0000
committer ocornut <omarcornut@gmail.com> Thu Mar 26 16:03:30 2015 +0000
tree 6310ccb5ce1b3df5f3e6a564a6b7f20d5510eaed
parent b04ee7e040bbada26539469e678fc96c1d7c9f58

Avoid negative clipping rectangle when collapsing windows, ImDrawList could still submit data #177

imgui.cpp

diff --git a/imgui.cpp b/imgui.cpp
index 5b2d6ba..144ce89 100644
--- a/imgui.cpp
+++ b/imgui.cpp
@@ -2098,6 +2098,7 @@
         cr = ImVec4(ImMax(cr.x, cur_cr.x), ImMax(cr.y, cur_cr.y), ImMin(cr.z, cur_cr.z), ImMin(cr.w, cur_cr.w));
     }
 
+    IM_ASSERT(cr.x <= cr.z && cr.y <= cr.w);
     window->ClipRectStack.push_back(cr);
     window->DrawList->PushClipRect(cr);
 }
@@ -3295,10 +3296,13 @@
 
     // Inner clipping rectangle
     // We set this up after processing the resize grip so that our clip rectangle doesn't lag by a frame
+    // Note that if our window is collapsed we will end up with a null clipping rectangle which is the correct behavior.
     const ImRect title_bar_rect = window->TitleBarRect();
     ImVec4 clip_rect(title_bar_rect.Min.x+0.5f+window->WindowPadding().x*0.5f, title_bar_rect.Max.y+0.5f, window->Rect().Max.x+0.5f-window->WindowPadding().x*0.5f, window->Rect().Max.y-1.5f);
     if (window->ScrollbarY)
         clip_rect.z -= style.ScrollbarWidth;
+    clip_rect.z = ImMax(clip_rect.x, clip_rect.z);
+    clip_rect.w = ImMax(clip_rect.y, clip_rect.w);
     PushClipRect(clip_rect);
 
     // Clear 'accessed' flag last thing
@@ -7360,6 +7364,8 @@
     draw_cmd.texture_id = texture_id_stack.empty() ? NULL : texture_id_stack.back();
     draw_cmd.user_callback = NULL;
     draw_cmd.user_callback_data = NULL;
+
+    IM_ASSERT(draw_cmd.clip_rect.x <= draw_cmd.clip_rect.z && draw_cmd.clip_rect.y <= draw_cmd.clip_rect.w);
     commands.push_back(draw_cmd);
 }
 

imgui.h

diff --git a/imgui.h b/imgui.h
index 60a6728..0e09024 100644
--- a/imgui.h
+++ b/imgui.h
@@ -45,7 +45,7 @@
 typedef int ImGuiColorEditMode;     // enum ImGuiColorEditMode_
 typedef int ImGuiMouseCursor;       // enum ImGuiMouseCursor_
 typedef int ImGuiWindowFlags;       // enum ImGuiWindowFlags_
-typedef int ImGuiSetCond;           // enum ImGuiSetCondition_
+typedef int ImGuiSetCond;           // enum ImGuiSetCond_
 typedef int ImGuiInputTextFlags;    // enum ImGuiInputTextFlags_
 struct ImGuiTextEditCallbackData;   // for advanced uses of InputText() 
 typedef int (*ImGuiTextEditCallback)(ImGuiTextEditCallbackData *data);