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
feat(shields): Add Eternal Keypad #1136
Conversation
I can't speak for other places, it is at least commercially available in Australia: https://customkbd.com/collections/complete-kits/products/eternal-keypad-kit |
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.
Sorry it took so long for the review! Re-request a review when you've addressed the feedback.
file_format: "1" | ||
id: eternal_keypad | ||
name: Eternal Keypad | ||
type: shield | ||
url: https://github.com/duckyb/eternal-keypad | ||
requires: [pro_micro] | ||
features: | ||
- keys |
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.
There should probably be another .zmk.yml file for _lefty
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.
Will do.
@@ -0,0 +1,54 @@ | |||
/* |
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.
I'm a bit confused at the differences in keymaps from this and the non-lefty variant (second layer totally blank here, slight differences in letters?)
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.
Unfortunately, I don't have a built version of the lefty, or an unbuilt pcb to test with. Since this is specifically a gaming board, and the pcb is reversible, you can cheat by using the default transform and flipping A/D (to preserve WASD), and LEFT/RIGHT, then all the other keys will be in the correct locations upside down.
I did make a mistake here by forgetting to swap LEFT/RIGHT and adding the extra BT control keys to the ARROW layer, so I'll fix that before review.
/ { | ||
chosen { | ||
zmk,kscan = &kscan0; | ||
zmk,matrix_transform = &default_transform; |
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.
Shouldn't this be the lefty transform?
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.
In an ideal world, yes. See the previous comment for details.
5a06678
to
d391fd6
Compare
@Nicell this PR is now ready for a re-review. |
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.
One minor nit-pick, should the display name match?
fcd98eb
to
e192a9f
Compare
Add Eternal Keypad gaming keypad.
Board/Shield Check-list
.zmk.yml
metadata file added&pro_micro
used in favor of&pro_micro_d/a
if applicable.conf
file has optional extra features commented out