Skip to content

Fix compatibility with ImGui 1.91.5+ and 1.92.6+#2

Merged
wirew0rm merged 1 commit intomasterfrom
fair-acc-compat
Mar 2, 2026
Merged

Fix compatibility with ImGui 1.91.5+ and 1.92.6+#2
wirew0rm merged 1 commit intomasterfrom
fair-acc-compat

Conversation

@RalphSteinhagen
Copy link
Member

  • based on upstream PR compat: use key-press helper and floor ImRect via ImFloor thedmd/imgui-node-editor#331.
  • replace removed ImRect::Floor() with FloorRect() helper and deprecated GetKeyIndex() with IsKeyPressedCompat() — both with backward-compatible version guards.
  • update vendored ImGui to v1.92.6-docking and fix examples
    • update external/imgui from v1.84 (thedmd fork) to v1.92.6-docking (stock ocornut/imgui) for smoke-testing the compatibility patch.
    • replace ancient imgui_impl_glfw/opengl3 backends with 1.92.6 versions.
    • fix ImTextureID type change (void* -> ImU64) in renderer and examples.
    • fix font atlas initialization for new texture management API.
    • fix obsoleted PushButtonRepeat -> PushItemFlag in widgets-example.
    • fix GetKeyIndex removal in blueprints-example.

Note: blueprints-example cannot build with stock ImGui — it requires ImGui::Spring/BeginHorizontal/BeginVertical from the thedmd/imgui fork.

Smoke test (repo has no unit-tests):

image

* based on upstream PR thedmd#331.
* replace removed ImRect::Floor() with FloorRect() helper and deprecated GetKeyIndex() with IsKeyPressedCompat() — both with backward-compatible version guards.
* update vendored ImGui to v1.92.6-docking and fix examples
  - update external/imgui from v1.84 (thedmd fork) to v1.92.6-docking (stock ocornut/imgui) for smoke-testing the compatibility patch.
  - replace ancient imgui_impl_glfw/opengl3 backends with 1.92.6 versions.
  - fix ImTextureID type change (void* -> ImU64) in renderer and examples.
  - fix font atlas initialization for new texture management API.
  - fix obsoleted PushButtonRepeat -> PushItemFlag in widgets-example.
  - fix GetKeyIndex removal in blueprints-example.

*Note: blueprints-example cannot build with stock ImGui — it requires
ImGui::Spring/BeginHorizontal/BeginVertical from the thedmd/imgui fork.*

Signed-off-by: Ralph J. Steinhagen <r.steinhagen@gsi.de>
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