Navigation does not lose focus when clicking on game background.

[rmitton]

Version/Branch of Dear ImGui:

Version: 1.77
Branch: docking

My Issue/Question:

Clicking outside the UI (i.e. on the underlying void or dockspace) used to release the navigation. However, in commit d31fe97 a change was made where it no longer handles the case where g.HoveredRootWindow == NULL. This causes the navigation focus to remain stuck, causing io.WantCaptureKeyboard to stay set and prevents the game from receiving any inputs. (until Esc is pressed at least)

[DomGries]

Personally I would prefer the current behavior with the newly arisen bugs fixed as this prevents keyboard/gamepad navigation being unusable after clicking on the background. Or at least maybe this can be made optional.

[batcholi]

A very simple fix is to capture the click callback and unfocus manually the imgui window via code...

// On Mouse Click Callback : 
if (!ImGui::IsAnyWindowHovered()) ImGui::SetWindowFocus(nullptr);

EDIT:
This does not work well because IsAnyWindowHovered() will return false while we are clicking on certain elements inside a window.... this removes the focus when we click on say a Checkbox... I would say this functionality is broken...

[ocornut]

Apologies for the issue and late reaction, will be looking at this.

[ocornut]

Pushed fix, this was incredibly dumb and thus equally puzzling and stressing to imagine that this bug went through.
We added some extra regression tests for it.

Minor note that:

Clicking outside the UI (i.e. on the underlying void or dockspace) used to release the navigation.

The "dockspace" part of that statement is wrong, unless you mean the transparent area of a "passthrough" dockspace but when you hover that you are not hovering the dockspace.

@domgho

Personally I would prefer the current behavior with the newly arisen bugs fixed as this prevents keyboard/gamepad navigation being unusable after clicking on the background.

There was a clear regression so we fixed it.
Some other threads discuss the possibility of adding more nav settings and i'll be happy to investigate them later (some applications are built over the premise that inputs should be shared between an underlying app and imgui stuff, some around the promise that navigation highlight should never ever be disabled, etc. we can work out alternate policies).