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

Added option to make Keystroke history follow mouse position #78

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

Conversation

elMuso
Copy link

@elMuso elMuso commented Feb 27, 2022

Please ignore the multiple commits, my pc can not run visual studio so i used github actions as my compiler (-.-). I was figuring out how c# worked. and i knew nothing about windows forms when i started editing the fork

Basically this program is almost perfect for my needs except that i tought it needed the option to make the keystrokes follow the mouse position, this is because i zoom a lot in videos that i make so having the keystrokes next to the mouse would make editing almost instantaneous.

Since the images follow the mouse already i just copied some code and added a textbox for it to work.
The new option can be found under "Show keystroke history" at the keystroke history tab. Toggling it restarts the app since i couldn't figure out how to preserve and restore the static position without doing it. But it's disabled by default

Initially i was going to make it for myself but seeing how it works i think it must be on the official binary. Some cleaning may be needed

@kmpm
Copy link
Contributor

kmpm commented Aug 10, 2022

The functionality is fine by me but you should perhaps do a git rebase and squash all those unnecessary commits.

@hectorC
Copy link

hectorC commented Nov 4, 2022

It would be great to have this included. This is the only function that is missing for my use case.

@luce80
Copy link

luce80 commented Aug 14, 2023

I think It would be great to have this included, too. Please make an exe with it.

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.

4 participants