-
Notifications
You must be signed in to change notification settings - Fork 109
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
Reviving this #46
Comments
@honzapatCZ I am interested. How do I contact you? |
@Pospelove One of us should fork the repo and we should continue the work there, My fork has dependabot configured for js, I know the changes there were working some time ago, but Iam not able to compile them now. |
@Pospelove happy to announce that I made my fork compile again |
@honzapatCZ Great. I am very interested in this lib. Thinking about dropping CEF in my project and use gelectron instead. I have been using Node native addons for years so we could be doing fantastic things (and maybe pull request back to the main repo). Consider adding me in Discord: Pospelov#3228 or write your ID in your-favorite-messenger. I think it would be reasonable to communicate, discuss ideas, etc via messengers, not gelectron issues. :) |
@hiitiger Can you elaborate why was the app separated like that? Iam thinking about merging the thread and process injection into node-ovhook (yes the 32 bit injection will be harder) and then merging this alltogether into electron-overlay making it only 2 separate projects game-overlay and electron-overlay |
@honzapatCZ I am following your progress with care! First of all thank you for making it compilable and update to set with modern toolset (even if I am looking to use your project to revive an old abandonned game using DirectX9....)
|
|
@honzapatCZ Not sure if you are still working on this, but there is a performance issue with the current hooks and mice that have a high polling rate. More info here #53 |
Interesting, not actively, but you are free to make suggestions or PRs |
/node-ovhook/src/main.cc
set this to true and injection into minimized full-screen windows will work |
Looks like @hiitiger lost interest, but I still think this is useful,
I know a bit about c++, so I can do small adjustments but understanding this source is pretty hard
First of all I think we should update it to the newest VS2019
Anyone wanna help?
The text was updated successfully, but these errors were encountered: