-
-
Notifications
You must be signed in to change notification settings - Fork 39.9k
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 handwired/dactyl_manuform/5x8 config #24726
base: master
Are you sure you want to change the base?
Conversation
keyboards/handwired/dactyl_manuform/5x8/keymaps/default/keymap.c
Outdated
Show resolved
Hide resolved
keyboards/handwired/dactyl_manuform/5x8/keymaps/default/keymap.c
Outdated
Show resolved
Hide resolved
keyboards/handwired/dactyl_manuform/5x8/keymaps/default/keymap.c
Outdated
Show resolved
Hide resolved
Thank you for comments to my code. I will fix my files and i will run on my keyboard for checking codes. If code will be work i will upload codes and push request for your control. |
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 rewrite all files
Hello. I need help. I've reviewed all the files and changes, added my comments, and tested them on my keyboard. The "Reopen deletion request" button is not active. Where can I read what needs to be done for the next step to merge? |
Reopening this PR as requested (in case you can't do that yourself due to some GitHub restrictions). |
change "soft_serial_pin": "D0" to "serial": { "pin": "D0" }
I see that all reviews have been resolved. Could you tell me if there's anything else I should do? |
Description
This PR adds support for the
handwired/dactyl_manuform/5x8
keyboard to QMK Firmware. The5x8
layout is a handwired version of the Dactyl Manuform with 5 rows and 8 columns per hand. It includes the following features:LAYOUT_5x8
.Types of Changes
Issues Fixed or Closed by This PR
Checklist