Fixing keyTyped() for dev-2.0 branch #7809
Conversation
|
Hi, @perminder-17 |
|
Hi @AhmedMagedC , Really thanks for raising your concerns, these discussions are really very appreciated in PROCESSING FOUNDATION so if anything goes wrong could be fixed :).
Here's the screenshot from the mdn docs Can you please confirm what you get when you hold any key for maybe 3-4 seconds, do you get If that's again false, can you please let me know what browser and version you using? here's the browser compatibility written " https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/repeat?utm_source=chatgpt.com " maybe due to compatibility issue you could be getting always false? Also, in your PR i.e. #7808 as I can see you are first setting In your PR, it's something like ; if (e.repeat) {
this._setProperty('isKeyRepeated', true); // isKeyRepeated is used further.
return;
}I am really sorry if I am not referring to the same thing you are talking about? Can you please let me know if I am clear to explain the use of |
|
Hi @perminder-17, the issue is that but when i tried to print the i tested on chrome & avast & Microsoft edge |
Okay, thanks for reporting. But I also checked on firefox and chrome (at linux) and everything was working correctly as expected. This looks like could be a browser specific issue. Someone previously had a similar issue with this. : https://stackoverflow.com/questions/53200698/onkeypress-repeat-always-returns-false-in-some-browsers Workaround could be, maybe we will move |

Resolves #7805
Changes:
Screenshots of the change:
PR Checklist
npm run lintpasses