Skip to content
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

Bump imgui version from v1.79 to v1.84.1. #11

Merged
merged 2 commits into from
Aug 22, 2021
Merged

Conversation

giladreich
Copy link
Owner

This PR updates ImGui submodule to v1.84.1 and required some changes due to API breaking changes and directory structure.

The CMake scripts and Visual Studio solutions have been updated accordingly.

Fixating on version v1.84.1 instead of v1.84 due to an issue noticed with that release.

@giladreich giladreich merged commit 91dd31c into master Aug 22, 2021
@giladreich giladreich deleted the imgui-update branch August 22, 2021 22:56
@ocornut
Copy link

ocornut commented Aug 23, 2021

Note: All your events handlers seem to be using tests like ImGui::IsWindowHovered(ImGuiHoveredFlags_AnyWindow) in order to decide whether to forward the event to underlying window. This is generally incorrect and will lead to variety of issues, you should use io.WantMouseCapture for this.

@giladreich
Copy link
Owner Author

giladreich commented Aug 23, 2021

Hey @ocornut,
Thanks a lot for the heads-up! I remember I ended up doing so as a workaround due to a Qt bug, since the rendering surface is an actual Qt Widget within the Qt window, causing some of the native key strokes not being forwarded. I'm not 100% sure, as it's been a while since I last looked into that. I'll double check your suggestion over the weekend and see if anything behavioral-wise has changed with Qt 5.15 / 6.

P.S: Thanks again for all your incredible work on ImGui, I'm so pleased to see the tables branch has been merged to master officially.

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