Closed JannesStroehlein closed 2 years ago
@JannesStroehlein Thank you very much for your contribution! I appreciate it 🙏🏽 I will start reviewing your code in the next few days
The typos should be fixed now.
Hey @JannesStroehlein,
I started to review your code and have some notes:
@JannesStroehlein hey, I reviewed all code and decided that the toggle feature is great, but the other feature should be split to a separate PR and needs more work.
If you want to continue on it just revert the last commit I pushed (47eea86a8274ff0b8b612759b14e2a538e1632ac)
Toggle Autoclicker and Auto stop on mouse movement
Currently I'm "playing" lots of CookieClicker and got repeatedly confused about the start/stop buttons - so I added a button that could do both. I also had a few autoclicker mishaps where I moved my mouse with the autoclicker enabled into another window with unintended results. That's why I added a option to stop clicking when the mouse moves.
Both functions work in hand with the existing code base. The new hotkey is editable in the settings window and also gets saved and unregistered. The hotkey id as well as the button label are saved in the Constants.cs file. The new checkbox's value also gets saved and loaded. I also added a slider to change the tolerance of the mouse movement detection.