ImWindow wrongly closes window when docking two ImWindows together

[joscha999]

When docking windows together ImGui.Begin will return false for the tab that is not currently selected.

ImWindow.cs will call close in this case which in turn sets IsShown false, see:
https://github.com/HexaEngine/Hexa.NET.ImGui.Widgets/blob/main/Hexa.NET.ImGui.Widgets/ImWindow.cs#L80

Commenting/Removing that line fixes the issue. Windows still close correctly via IsShown and the following if checking/closing it.

[JunaMeinhold]

Hi,
I never had any issues with it and i used docking a lot, could you give me a more in depth example?

[joscha999]

Hi Juna, steps to reproduce this issue (at least on my end):

• Create a new Console project (default settings in VS/dotnet new, use .Net 10)
• Install Hexa.NET.KittyUI Version 1.1.13
• Use the following code:

// See https://aka.ms/new-console-template for more information
using Hexa.NET.ImGui;
using Hexa.NET.ImGui.Widgets;
using Hexa.NET.KittyUI;

AppBuilder builder = new();
builder.AddWindow<TestWindow1>();
builder.AddWindow<TestWindow2>();
builder.Run();

internal class TestWindow1 : ImWindow {
    public override string Name => "TestWindow1";
    public override void DrawContent() {
        ImGui.Text("Hello World");
    }
}

internal class TestWindow2 : ImWindow {
    public override string Name => "TestWindow2";
    public override void DrawContent() {
        ImGui.Text("Hello World");
    }
}

• Run the app
• Drag one of the two Windows into the center dock space of the other window:

Expected result: We get one window with two tabs (like the preview shows in the screenshot).
Actual result: The window we drop onto gets closed and only the dragged one remains.

Edit: Probably could have worded the title/initial issue better - it's explicitely dropping two windows "on top of each other" - not the best with words sorry - hope the steps to reproduce are clear/you can reproduce it D:

[JunaMeinhold]

Hi, thanks for finding out, i currently don't have any spare time, could anyone please make a PR.