Skip to content

Move PopStyleVar call to balanced spot - #32

Open
kavika13 wants to merge 1 commit into
Gyrth:drika_hotspotfrom
kavika13:patch-2
Open

kavika13 wants to merge 1 commit into
Gyrth:drika_hotspotfrom
kavika13:patch-2

Conversation

@kavika13

@kavika13 kavika13 commented Oct 1, 2019

Copy link
Copy Markdown

Debug builds (not available to players, just wolfire devs) give warnings if they aren't balanced properly inside and outside begin/end pairs. It could potentially cause weird behavior, crashes, or other UIs to do the wrong thing if they're not properly balanced.

Debug builds (not available to players, just wolfire devs) give warnings if they aren't balanced properly inside and outside begin/end pairs. It could potentially cause weird behavior, crashes, or other UIs to do the wrong thing if they're not properly balanced.
ImGui_SetNextWindowSize(vec2(600.0f, 400.0f), ImGuiSetCond_FirstUseEver);
ImGui_SetNextWindowPos(vec2(100.0f, 100.0f), ImGuiSetCond_FirstUseEver);
ImGui_Begin("Drika Hotspot " + (show_name?" - " + display_name:"" + this_hotspot.GetID()) + "###Drika Hotspot", show_editor, ImGuiWindowFlags_MenuBar);
ImGui_PopStyleVar();

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At line 535 is also a ImGui_PopStyleVar(); does that need to be removed as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants