-
-
Notifications
You must be signed in to change notification settings - Fork 39.7k
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 support for KBDPAD MKII #6034
Conversation
* represents the switch matrix. | ||
*/ | ||
#define LAYOUT( \ | ||
K00, K01, K02, K03, \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
curious why you decided to go Kxy here and for the kbd8x mkii you went with Kxyz.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking back at it, I have no idea. Probably either extreme fatigue, or too long of a gap between the pad and 8x's revised PCB that I forgot how I labeled it on the 8x.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you wanted to add community layout support?
Co-Authored-By: Elliot Powell <[email protected]> Co-Authored-By: Drashna Jaelre <[email protected]>
Co-Authored-By: Drashna Jaelre <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
Thanks!
* Build some testing keymaps * Match naming convention to 8x * Add configurator json * Forgot to build the readme * Apply suggestions from code review Co-Authored-By: Elliot Powell <[email protected]> Co-Authored-By: Drashna Jaelre <[email protected]> * Update keyboards/kbdfans/kbdpad_mk2/rules.mk Co-Authored-By: Drashna Jaelre <[email protected]>
* Build some testing keymaps * Match naming convention to 8x * Add configurator json * Forgot to build the readme * Apply suggestions from code review Co-Authored-By: Elliot Powell <[email protected]> Co-Authored-By: Drashna Jaelre <[email protected]> * Update keyboards/kbdfans/kbdpad_mk2/rules.mk Co-Authored-By: Drashna Jaelre <[email protected]>
* Build some testing keymaps * Match naming convention to 8x * Add configurator json * Forgot to build the readme * Apply suggestions from code review Co-Authored-By: Elliot Powell <[email protected]> Co-Authored-By: Drashna Jaelre <[email protected]> * Update keyboards/kbdfans/kbdpad_mk2/rules.mk Co-Authored-By: Drashna Jaelre <[email protected]>
* Build some testing keymaps * Match naming convention to 8x * Add configurator json * Forgot to build the readme * Apply suggestions from code review Co-Authored-By: Elliot Powell <[email protected]> Co-Authored-By: Drashna Jaelre <[email protected]> * Update keyboards/kbdfans/kbdpad_mk2/rules.mk Co-Authored-By: Drashna Jaelre <[email protected]>
Description
Adds support for the KBDPAD MKII numpad.
Types of Changes
Issues Fixed or Closed by This PR
Checklist