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

Wrapping existing variables with ImGui widgets #138

Open
calvertdw opened this issue Oct 8, 2022 · 1 comment
Open

Wrapping existing variables with ImGui widgets #138

calvertdw opened this issue Oct 8, 2022 · 1 comment
Labels
feat New feature or request question Further information is requested

Comments

@calvertdw
Copy link
Contributor

calvertdw commented Oct 8, 2022

Currently, a lot of the API accepts ImFloat or float[] and the cooresponding types. It does this so ImGui can have a setter and a getter to that variable. However, the ImFloat, etc. are implemented as final class. It would be more convenient if the ImGui API instead accepted an interface instead, with get, set, etc. such that one could wrap existing variables with ImGui widgets. I believe this is frequently done in the C++ version and one of the best ways to use ImGui.

@SpaiR what do you think about this? For now I will just sync manually before and after the ImGui calls.

@SpaiR
Copy link
Owner

SpaiR commented Oct 8, 2022

🤔 Sounds like a good idea. Can you provide an example of how the API may look?

@SpaiR SpaiR added feat New feature or request question Further information is requested api labels Dec 2, 2022
@SpaiR SpaiR removed the api label Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants