WIP Menus: Fixed remaining inconsistency with stacks of popups. Activating/closing a menu close the parent popup. Added test case. (#126)

This commit is contained in:
ocornut 2015-05-26 20:33:48 +01:00
parent 90cf77b191
commit 3a6e6645e6
2 changed files with 22 additions and 16 deletions

View File

@ -3072,32 +3072,35 @@ void ImGui::OpenPopup(const char* str_id)
g.OpenedPopupStack.back() = ImGuiPopupRef(id, window, window->GetID("##menus"));
}
static void ClosePopupToLevel(int remaining)
{
ImGuiState& g = *GImGui;
if (remaining > 0)
FocusWindow(g.OpenedPopupStack[remaining-1].Window);
else
FocusWindow(g.OpenedPopupStack[0].ParentWindow);
g.OpenedPopupStack.resize(remaining);
}
static void ClosePopup(const char* str_id) // not exposed because 'id' scope is misleading
{
ImGuiState& g = *GImGui;
ImGuiWindow* window = GetCurrentWindow();
const ImGuiID id = window->GetID(str_id);
if (IsPopupOpen(id))
g.OpenedPopupStack.resize(g.CurrentPopupStack.size());
ClosePopupToLevel((int)g.CurrentPopupStack.size());
}
// Close the popup we have begin-ed into.
void ImGui::CloseCurrentPopup()
{
ImGuiState& g = *GImGui;
int popup_idx = (int)g.CurrentPopupStack.size() - 1;
if (popup_idx < 0 || popup_idx > (int)g.OpenedPopupStack.size() || g.CurrentPopupStack[popup_idx].PopupID != g.OpenedPopupStack[popup_idx].PopupID)
return;
if (g.CurrentWindow->PopupID == g.OpenedPopupStack[popup_idx].PopupID && g.Windows.size() > 1)
FocusWindow(g.Windows[g.Windows.size()-2]);
g.OpenedPopupStack.resize(popup_idx);
}
static void CloseCurrentMenus()
{
// Close all popups
// FIXME-MENUS: invalid for popup->menus with current BeginMenu() scheme
ImGuiState& g = *GImGui;
g.OpenedPopupStack.resize(0);
while (popup_idx > 0 && g.OpenedPopupStack[popup_idx].Window && (g.OpenedPopupStack[popup_idx].Window->Flags & ImGuiWindowFlags_ChildMenu))
popup_idx--;
ClosePopupToLevel(popup_idx);
}
static void ClearSetNextWindowData()
@ -7357,9 +7360,7 @@ static bool SelectableEx(const char* label, bool selected, const ImVec2& size_ar
if (flags & ImGuiSelectableFlags_Disabled) ImGui::PopStyleColor();
// Automatically close popups
if (pressed && (window->Flags & ImGuiWindowFlags_ChildMenu) && !(flags & ImGuiSelectableFlags_DontClosePopups))
CloseCurrentMenus();
else if (pressed && (window->Flags & ImGuiWindowFlags_Popup) && !(flags & ImGuiSelectableFlags_DontClosePopups))
if (pressed && !(flags & ImGuiSelectableFlags_DontClosePopups) && (window->Flags & ImGuiWindowFlags_Popup))
ImGui::CloseCurrentPopup();
return pressed;
}
@ -10557,6 +10558,11 @@ void ImGui::ShowTestWindow(bool* opened)
{
for (int i = 0; i < IM_ARRAYSIZE(names); i++)
ImGui::MenuItem(names[i], "", &toggles[i]);
if (ImGui::BeginMenu("Sub-menu"))
{
ImGui::MenuItem("Click me");
ImGui::EndMenu();
}
ImGui::EndPopup();
}
ImGui::EndPopup();

View File

@ -168,7 +168,7 @@ namespace ImGui
IMGUI_API void OpenPopup(const char* str_id); // mark popup as open. close childs popups if any. will close popup when user click outside, or activate menu items, or CloseCurrentPopup() is called within a BeginPopup/EndPopup block.
IMGUI_API bool BeginPopup(const char* str_id); // return true if popup if opened and start outputting to it. only call EndPopup() if BeginPopup() returned true!
IMGUI_API void EndPopup();
IMGUI_API void CloseCurrentPopup(); // close the popup we have begin-ed into
IMGUI_API void CloseCurrentPopup(); // close the popup we have begin-ed into. clicking on a MenuItem or Selectable automatically close the current popup.
// Layout
IMGUI_API void BeginGroup(); // once closing a group it is seen as a single item (so you can use IsItemHovered() on a group, SameLine() between groups, etc.