From 74bbfcfaa668fd0e35b41c04b18f7a15f5419833 Mon Sep 17 00:00:00 2001 From: ocornut Date: Wed, 29 Jun 2016 10:07:36 +0200 Subject: [PATCH] ImGuiListClipper: fixed automatic-height calc path dumbly having user display element 0 twice (#661, #716) First bug out of two. Was easily visible using e.g. 50% alpha text. --- imgui.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/imgui.cpp b/imgui.cpp index 904cef81..a9f9dbaa 100644 --- a/imgui.cpp +++ b/imgui.cpp @@ -1680,8 +1680,9 @@ bool ImGuiListClipper::Step() if (ItemsCount == 1) { ItemsCount = -1; return false; } float items_height = ImGui::GetCursorPosY() - StartPosY; IM_ASSERT(items_height > 0.0f); // If this triggers, it means Item 0 hasn't moved the cursor vertically - ImGui::SetCursorPosY(StartPosY); // Rewind cursor so we can Begin() again, this time with a known height. - Begin(ItemsCount, items_height); + Begin(ItemsCount-1, items_height); + DisplayStart++; + DisplayEnd++; StepNo = 3; return true; }