Skip to content
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

Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

Conversation

iliorik
Copy link

@iliorik iliorik commented Dec 17, 2024

Description

This PR adds support for the handwired/dactyl_manuform/5x8 keyboard to QMK Firmware. The 5x8 layout is a handwired version of the Dactyl Manuform with 5 rows and 8 columns per hand. It includes the following features:

  • Default keymap with support for ANSI layout.
  • Configurable layout using LAYOUT_5x8.
  • Basic documentation for building and flashing the firmware.

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Keyboard (addition or update)
  • Keymap/layout (addition or update)
  • Documentation

Issues Fixed or Closed by This PR

Checklist

  • My code follows the code style of this project: C, Python.
  • I have read the PR Checklist document and have made the appropriate changes.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@iliorik
Copy link
Author

iliorik commented Dec 22, 2024

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.

@iliorik iliorik closed this Dec 22, 2024
@waffle87 waffle87 mentioned this pull request Dec 23, 2024
9 tasks
Copy link
Author

@iliorik iliorik left a 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

@iliorik
Copy link
Author

iliorik commented Dec 24, 2024

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?

@sigprof
Copy link
Contributor

sigprof commented Dec 24, 2024

Reopening this PR as requested (in case you can't do that yourself due to some GitHub restrictions).

@sigprof sigprof reopened this Dec 24, 2024
@iliorik iliorik requested a review from waffle87 December 24, 2024 07:44
@iliorik
Copy link
Author

iliorik commented Dec 26, 2024

I see that all reviews have been resolved. Could you tell me if there's anything else I should do?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants