-
Notifications
You must be signed in to change notification settings - Fork 256
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
add wheel effect #338
add wheel effect #338
Conversation
Wow, works great and looks so cool!! 👍 I really regret that I put the col/row properties in the feature config and not to the device itself. It would have saved you some work/time I guess. I'm sorry about this. I'll happily clean that after we merge your PR. Edit: This PR will also fix #57 |
Apologies for the delay guys. Will merge and release! |
Is there a reason for keyboards having 'wheel' effects @stevenliuyi?. (https://github.com/stevenliuyi/razer-macos/blob/296abf2e350d798954e4ef43da38709b1bb114bc/src/devices/huntsman_elite.json) It shows up as an option for my keyboard but its greyed out with 'Wheel (missing rows, col config)' Will release for now though. |
I've made the latest release a 'pre-release'. Not exactly sure how to fix it until I have clarification from the others. |
I couldn't see you screenshot could you please reupload or let me know what device it was? Thanks |
My bad 🤦. I thought wheel effect was for mice, makes sense now |
Yes, it's only for keyboards. A side note: I don't have a Windows PC right now, so I can't use Synapse 3 to set the official wheel effect on my keyboard. My implementation is just based on the official Youtube video (https://www.youtube.com/watch?v=KGURYdPXJe4), so it may well not be a very accurate replica. |
Do you still have the wheel effect greyed out with Huntsman Elite? It seems weird since the configuration is already added on https://github.com/1kc/razer-macos/blob/master/src/devices/huntsman_elite.json |
Actually, I just found out the issue. I'll fix it soon. |
Sounds good @stevenliuyi. I think this one might be related also #342 |
It should be fixed by #343 |
That PR #343 doen't include any changes for Blackwidow V3, you think it will still fix it? |
It seems that the configuration for Blackwidow V3 has been missing since the last release. I just added the configuration, so it should be good now. |
Ah cheers @stevenliuyi, I will release it now. |
@avigucci
Right now the saturation is set to be 70%. I think increasing the saturation should help. Maybe someone can test to see what would be the best value that matches the official effect? Perhaps it should be just 100%? |
Just tested it on my huntsman, it works and is indeed very cool! |
@1kc |
Add wheel effect to resolve #325