-
Notifications
You must be signed in to change notification settings - Fork 105
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
Better packaging? #55
Comments
Hi! I would be interested in this fork / pull request with regular typescript support. Could you push your modified branch or create a PR? Thanks in advance! |
I would love to see this. A basic example or two showing how to set a boilerplate project up using different frameworks would be great. |
This is what prevents me from using imgui-js, actually. |
This is such a good project that's a shame that its setup is so complicated for starters. I gave up after trying to integrate it to a TypeScript project... |
As of right now, this implementation feels a little too oriented around building and showing the demo. I am not sure how many people use System.js, but I do think some better defaults around the module would help a lot of people. Just so one could do a typical
import ImGui from "imgui-js"
, evenimport {ImGui, ImGui_Impl} from "imgui-js"
out of the box.I can certainly take this on and do a PR, (I already have a fork with the minimal changes needed to do a regular import) but just wanted to see if the library maintainers have any interest in it, because it would be some substantial, breaking changes depending on how people are using the library.
The text was updated successfully, but these errors were encountered: