Browse Source

Closing the focused window restore focus to the first active root window in descending z-order (part of #727)

ocornut 9 years ago
parent
commit
19d02becef
1 changed files with 12 additions and 6 deletions
  1. 12 6
      imgui.cpp

+ 12 - 6
imgui.cpp

@@ -2250,14 +2250,11 @@ void ImGui::NewFrame()
                 window->SizeFull *= scale;
                 window->SizeFull *= scale;
             }
             }
         }
         }
-        else
+        else if (!(window->Flags & ImGuiWindowFlags_NoScrollWithMouse))
         {
         {
             // Scroll
             // Scroll
-            if (!(window->Flags & ImGuiWindowFlags_NoScrollWithMouse))
-            {
-                const int scroll_lines = (window->Flags & ImGuiWindowFlags_ComboBox) ? 3 : 5;
-                SetWindowScrollY(window, window->Scroll.y - g.IO.MouseWheel * window->CalcFontSize() * scroll_lines);
-            }
+            const int scroll_lines = (window->Flags & ImGuiWindowFlags_ComboBox) ? 3 : 5;
+            SetWindowScrollY(window, window->Scroll.y - g.IO.MouseWheel * window->CalcFontSize() * scroll_lines);
         }
         }
     }
     }
 
 
@@ -2275,6 +2272,15 @@ void ImGui::NewFrame()
         window->Accessed = false;
         window->Accessed = false;
     }
     }
 
 
+    // Closing the focused window restore focus to the first active root window in descending z-order
+    if (g.FocusedWindow && !g.FocusedWindow->WasActive)
+        for (int i = g.Windows.Size-1; i >= 0; i--)
+            if (g.Windows[i]->WasActive && !(g.Windows[i]->Flags & ImGuiWindowFlags_ChildWindow))
+            {
+                FocusWindow(g.Windows[i]);
+                break;
+            }
+
     // No window should be open at the beginning of the frame.
     // No window should be open at the beginning of the frame.
     // But in order to allow the user to call NewFrame() multiple times without calling Render(), we are doing an explicit clear.
     // But in order to allow the user to call NewFrame() multiple times without calling Render(), we are doing an explicit clear.
     g.CurrentWindowStack.resize(0);
     g.CurrentWindowStack.resize(0);