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

Assert related to SetCursorPos fires in Demo app #160

Closed
Damon3000s opened this issue Jan 9, 2025 · 2 comments
Closed

Assert related to SetCursorPos fires in Demo app #160

Damon3000s opened this issue Jan 9, 2025 · 2 comments

Comments

@Damon3000s
Copy link
Collaborator

While running the Demo project there are asserts from ImGui.NET about SetCursorPos being used incorrectly. This was recently introduced when the ImGui.NET dependency was updated.

The assert is: "Code uses SetCursorPos()/SetCursorScreenPos() to extend window/parent boundaries. Please submit an item e.g. Dummy() to validate extent."

The code can be found here: https://github.com/ocornut/imgui/blob/master/imgui.cpp#L10084
The linked issue ca be found here: ocornut/imgui#5548

@Damon3000s
Copy link
Collaborator Author

I believe the issue is how the grid advances the cursor. It always advances to the top left of the next cell for drawing the content but I think it also advances one more time even after the final cell has been drawn. This is what I will investigate first.

@matt-edmondson
Copy link
Contributor

Fixed by #161

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

2 participants