-
Notifications
You must be signed in to change notification settings - Fork 125
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
Emscripten fix #116
base: master
Are you sure you want to change the base?
Emscripten fix #116
Conversation
Hey, Thanks for your PR. Have you seen the |
@Daandelange I tried the
But I am sure it will work with the right changes (because it does work well with the master branch). |
Hi, thanks for giving it a try. :) Look in ofxImGuiConstants.h if the correct macros are defined. |
Hi, did you make any progress on this ? |
Hi, sorry. I did not make any progress with that. But for another project I created SWIG bindings for ofxImGui (not related to the Emscripten stuff). I will come back to the issue, eventually (I make a short summer programming break at the moment...). |
I did look into this as I had some GL errors in one of the builds with particular Makefile settings for the new libraries and Version of Emscripten. If possible ImGui update to latest just out this week and I'll look at the bindings as it solves |
@danoli3 Would you like me to update ImGui or are you on it ? (see Updating ImGui) |
Feel free to jump in and update it to the latest, commit any progress if you get stuck but always can look past ES 1 for the moment |
OK, I just updated ImGui to v 1.91.0 which seems to introduce some glfw+emscripten fixes, is that what you're looking after ? |
I might be a little slow at the moment with programming, because I enjoy the summer, but I will check it soon. Thank you. I will try it in the next days. |
No hurries, take your time and enjoy ;) |
With adding
#include "stack"
toImHelpers.h
(and a few small additional changes) it works also with Emscripten.