Skip to content

bump ImGui to 1.91 #248

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

bump ImGui to 1.91 #248

wants to merge 1 commit into from

Conversation

hzqst
Copy link

@hzqst hzqst commented Jun 29, 2025

bump ImGui to 1.91, see #246 , io.KeyMap[] and io.KeysDown[] are removed, according to ImGui CHANGELOG.txt

and explicitly indicate shader language for ImGui VS & ImGui PS, to avoid getting error on shader compilation.

ThirdParty/imgui needs to be sync.

…sDown[] are removed (obsoleted February 2022). according to ImGui CHANGELOG.txt

and explicitly indicate shader language for ImGui VS & ImGui PS, to avoid getting error on shader compilation.
@hzqst hzqst requested a review from TheMostDiligent as a code owner June 29, 2025 05:09
@TheMostDiligent
Copy link
Contributor

How this change is supposed to work with the current version of ImGui?

@hzqst
Copy link
Author

hzqst commented Jun 30, 2025

How this change is supposed to work with the current version of ImGui?

I merged this patch with DiligentTools in a fresh cloned DiligentEngine and replace ThirdParty/imgui with the official one (v1.91) , that's all.

@TheMostDiligent
Copy link
Contributor

This change can't be merged. First of all, it breaks the build because it does not update the ImGui.
But more importantly, ImGui update is problematic. There were some issues on MacOS.
ImGui update needs to be thoroughly tested on all platforms. Plus your issues with code produced by GCC are also a concern.

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