-
Notifications
You must be signed in to change notification settings - Fork 21
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
Thank you #9
Comments
@javl You are super welcome! I'm sorry that I missed this issue before. I'm doing most of my work on sourcehut nowadays, and I guess I missed the email from GitHub. Anyway, it's cool that you were able to adapt this to the mac version of the keyboard! I was wondering what you'd think of combining the key table that you have with the one here. It seems that you had to change the locations of could instead read: /* B */ 0, 0, 0, NEXT, PREV, NEXT, PREV, 0, EJCT, 0, 0, 0, 0, 0, 0, 0, With that modification, I think that this driver would simply work for the mac keyboard in addition to the Windows and Pro versions of the keyboard. If that's the case, would you be interested in submitting a PR to add support for the mac edition? I really appreciate the attribution that you gave in your fork's README by the way. |
Sorry, I actually stopped using that keyboard a while ago. Couldn't really get used to it (partly because it was too worn down). |
Also not an issue but thank you, this was exactly my use case (Freestyle 2 on Debian) and it works really well. Sometimes requires a reboot after a kernel update, I think? Can try to reproduce and test. Thank you! |
I'm very glad it's useful to you.
Hmm. That would be unexpected. Please do try to gather more info about that and put it in a new issue. |
Not an issue, but a thank you for creating and sharing this driver!
I just got a second hand Freestyle (not the v2, and a mac version actually) and was kind of bummed the media keys didn't work on Debian. I've forked your project and adapted the key mapping to work with this keyboard (apparently this older model uses some other key codes) and all of them work perfectly now (now to remap those silly copy-paste shortcut buttons somehow).
The text was updated successfully, but these errors were encountered: