Skip to content

[Keymap] Drashna's split updates #13350

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

Merged
merged 34 commits into from
Jul 1, 2021
Merged

[Keymap] Drashna's split updates #13350

merged 34 commits into from
Jul 1, 2021

Conversation

drashna
Copy link
Member

@drashna drashna commented Jun 26, 2021

Description

Removes the custom transport in favor of of the extensibly split code.

This works fine for most boards, but not if a pointing device is being used and reports carried over the split transport.

Also included are a lot of bug fixes, edge case resolution, and formatting cleanup.
Also, moves the dactyl manuform with a trackball into "tractyl manuform" folder, adds a 4x6 version, and adds an actual default.

Types of Changes

  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)

Checklist

  • [x} 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).

@drashna drashna requested a review from a team June 26, 2021 06:41
@drashna drashna merged commit b7cf9a8 into qmk:develop Jul 1, 2021
nhongooi pushed a commit to nhongooi/qmk_firmware that referenced this pull request Dec 5, 2021
Co-authored-by: Ryan <fauxpark@gmail.com>
BorisTestov pushed a commit to BorisTestov/qmk_firmware that referenced this pull request May 23, 2024
Co-authored-by: Ryan <fauxpark@gmail.com>
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.

2 participants