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

Window not updating during resizing #71

Open
learn-more opened this issue Feb 14, 2024 · 10 comments
Open

Window not updating during resizing #71

learn-more opened this issue Feb 14, 2024 · 10 comments

Comments

@learn-more
Copy link

While the window is being resized, it is not updating.
This leads to artifacts on the right and bottom of the screen.

Is there interest in a PR fixing this?

@andrei-lz
Copy link

andrei-lz commented Feb 24, 2024

I believe this is a Windows issue rather than a Walnut issue as this happens to a wide variety of applications using ImGui or even Qt in my experience.

I also don't see it happen on Linux builds of the same applications

@learn-more
Copy link
Author

I believe this is a Windows issue rather than a Walnut issue as this happens to a wide variety of applications using ImGui or even Qt in my experience.

I also don't see it happen on Linux builds of the same applications

This is not a Windows issue, but an issue in the default ImGui implementation.
See ocornut/imgui#6461 for more info.

@andrei-lz
Copy link

Oh damn, you seem to be correct, my mistake

@learn-more
Copy link
Author

@TheCherno Is Walnut derived from Hazel's code or the other way around?

@lknknm
Copy link

lknknm commented Jul 19, 2024

@learn-more Hello, were you able to fix this? I tried updating my ImGui submodule inside Walnut to see if a newer build would help but I wasn't able to achieve it. Resizing still has this odd behaviour. If you have a PR to commit, I would be happy to give it a look to implement it on my project.

@learn-more
Copy link
Author

learn-more commented Jul 20, 2024

@learn-more Hello, were you able to fix this? I tried updating my ImGui submodule inside Walnut to see if a newer build would help but I wasn't able to achieve it. Resizing still has this odd behaviour. If you have a PR to commit, I would be happy to give it a look to implement it on my project.

I fixed this locally, but since this repository does not seem active at all I assumed the project to be abandoned.

It needs both a fix in their fork of GLFW (to do with invalidating window contents), and some changes in the Dear ImGui code that I sent as a PR to Dear ImGui:

This is not a Windows issue, but an issue in the default ImGui implementation. See ocornut/imgui#6461 for more info.

If you really need this I can probably dig it up from my old machine, but that will take a while.

From memory the patch in glfw was changing this code to return 0:

https://github.com/TheCherno/glfw/blob/026a148d7dd78d597de380c4e77ca0869f0ceaab/src/win32_window.c#L1046-L1048

@lknknm
Copy link

lknknm commented Jul 21, 2024

@learn-more Thank you for the quick reply!
So yeah, yesterday I was updating my Walnut submodule to point to the updated version of the ImGui, but I see your PR wasn't merged so the fix wouldn't work. Plus, Walnut uses Vulkan whilst I see you were using DX9, so I didn't know how to translate the knowledge. Should I just merge your PR and will it be fine even if Walnut uses Vulkan?

Also, changing the Walnut ImGui fork to my own fork of the updated ImGui repository seemed to break the custom Titlebar. Spent some hours trying to fix it with no success. I will probably try to reverse changes and apply your code on top of the original code from StudioCherno. Let's see.

Thank you once again!

@VagueLobster
Copy link

VagueLobster commented Jul 29, 2024

No not abandoned, a sub-project of a much larger private project... so it's not their main focus but a nice gift... plenty of us here overseeing it. Welcome aboard.

Walnut was released maybe not even a year ago now? It's not super active as most aren't very familiar yet and it has some limitations and is rather niche considering it helps junior developers but still requires some professional level coding knowledge to do much with really. ... it's always going to be different from other applications simply because it is, in almost every way.

Correction: Walnut was initially released in 2022, and the last update on the dev branch was from July 17th 2023 👍

learn-more added a commit to learn-more/Walnut that referenced this issue Aug 5, 2024
learn-more added a commit to learn-more/Walnut that referenced this issue Aug 17, 2024
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

No branches or pull requests

5 participants
@learn-more @VagueLobster @andrei-lz @lknknm and others