-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Can't edit any text input fields on my page with Emscripten's .js file loaded. #22947
Comments
I just confirmed that the problem is also there if my em++ target is the .html file (edited to add an input field) rather than the .js file with the custom html. |
@sbc100 Thanks. When you say "this will be fixed by," do you mean there's a fix for it somewhere in the thread, or do you mean Emscripten is planning on fixing the issue but hasn't yet? |
@inhahe I created a PR because I had a similar problem. It has not been merged yet. If you follow the relevant PR, your problem should be solved when it is merged. |
@uysalibov Oh, I see, thanks. Can you tell me how to follow the PR? I can't find a button to do that. |
@uysalibov Hmm, I did Edit: I just tried it again and got |
There is a subscribe button on the rightside.
Probably this error is related about gh client. I don't think it's about related PR |
Thanks. The problem was I was on https://github.com/emscripten-core/emscripten/pull/22879/files instead of #22879.
Hmm, is there another way to pull the commit than gh? |
Oh, I think #22879 only fixes the issue for GLFW and SDL programs, not SDL2 programs. @Daft-Freak do you know if SDL2 suffers from the same issue, and can/should we apply a similar fix there? |
Doing the same in SDL2 would be more difficult as we don't have access to There are the workarounds of disabling EDIT: That would need to be |
@sbc100 Not sure if this is a problem with your code or if I did something wrong, but I did
After having copied your modified file into emsdk/upstream/emscripten/src/ I may have done a |
You shouldn't copy all modified file. You just need to change related lines. There may be other pr's merged before this pr. You lose them |
Oh, I get it, thanks. That makes sense. |
Please include the following in your bug report:
Version of emscripten/emsdk:
Full link command and output with
-v
appended:So, here's a minimal html file that exhibits this behavior. I didn't even include the canvas object or anything for simplicity, but rest assured the problem persists even if it's a full proper web app.
I tried it using two different compiled .cpp files, one of them was rather small.
Please help me, Emscripten's js file is way too complicated for me to debug. ;p
The text was updated successfully, but these errors were encountered: