Skip to content
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

Fixes crash and layout, add log switch #6

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

danch15
Copy link

@danch15 danch15 commented Jan 17, 2021

  1. Fixes crash when device description registry key doesn't exist
  2. Fixes the KeyboardRedirectorForm and ActionPropertiesDialog Layout
  3. Add feature: enable and disable log to file. Update to Visual Studio 2019. Add feature: enable and disable log to file. #5 (by Starfer)
  4. Add Setting Button to open setting folder

2. Fixes the KeyboardRedirectorForm and ActionPropertiesDialog Layout
3. Add feature: enable and disable log to file. natevoci#5 (by Starfer)
@danch15
Copy link
Author

danch15 commented Jan 17, 2021

@natevoci
Copy link
Owner

Hi @danch15. Sorry it has taken me so long to get to look at this. I have just now merged Starfer's PR into master, so this PR conflicts now. It would be great if you could rebase your commits.

Also, could you please provide comments in english in your 2nd and 3rd commits, and some more information about the 3rd commit would be helpful. What is the "SCT.DC" process that you're sending messages to? And was there a specific reason you needed to add the LOG compilation flag rather than just uncheck the Log setting Starfer added?

@danch15
Copy link
Author

danch15 commented Aug 11, 2021

Hi @danch15. Sorry it has taken me so long to get to look at this. I have just now merged Starfer's PR into master, so this PR conflicts now. It would be great if you could rebase your commits.

Also, could you please provide comments in english in your 2nd and 3rd commits, and some more information about the 3rd commit would be helpful. What is the "DC" process that you're sending messages to? And was there a specific reason you needed to add the LOG compilation flag rather than just uncheck the Log setting Starfer added?

  1. Sorry, the process "sending messages" is send message to other application, you can just knock it off.
  2. 升级到.NET4.0, 优化部分逻辑,移除窗体最小化控制的定时器 -> Upgrade framework to. Net 4.0, optimize some logic and remove the timer of form minimization control (I use the better way to control windows minimization)
  3. 请采用Release编译,默认条件RAWINPUT,!LOG(增加代码中日志代码执行开关,并且加入根据指定键盘ID发送消息的开关)-> Please compile with release version. The default condition is RAWINPUT,!LOG (add the log code execution switch in the code, and add the switch to send messages according to the specified hard disk ID) This 3rd not need to submit to your branch, just my custom version... knock it off too.
  4. At last, there is a problem that is seems in the windows 10 20H2 or newer version, when your run this KeyboardRedirector app, and set Capture All Keys from this keyboard or some keys, than open the Windows Setting Form(WinKey+I), sometimes the capture is not work, you can input the key into anywhere. I hope you can fix it.
  5. "https://github.com/aelij/RawInputProcessor" I use this project to capture the keyboard's key in my wpf application and run the PTT(Push-To-Talk) method, and used your KeyboardRedirector tools to capture the key in other applications. But in windows 10 20H2 or newer version, the RawInputProcessor is not work too, can you fix it, it seems like the windows message is change with the windows version update. can you help me deal with it.
  6. I will rebase my commits when I have time, by the way, thanks for for your tools. My endlish is bad, please forgive me. (ˉ▽ˉ;)...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants